Re: [PATCH] psql: Fix REPACK index completion after a column list

Hi Chao,

I found a problem:
> ```
> evantest=# repack t (a)USING INDEX ;
> ERROR: ANALYZE option must be specified when a column list is provided
> ```
> After I typed in “(a)”, a tab typing auto filled in “USING INDEX”, but you
> can see the error. A column list is only allowed with ANALYZE specified. So,
> ```
> Matches("REPACK", MatchAny, "(*)")
> ```
> is questionable. It matches to above example.
>

Thanks for catching this. In v3, I've removed the new `USING INDEX`
completion directly after a column list, preserving the existing
behaviour there. I've also removed completion after `REPACK t (a) USING`
and `REPACK t (a) USING INDEX`, since a column list requires `ANALYZE`
to be enabled in the command's option list.

Checking whether that option list enables `ANALYZE` adds
enough complexity that I'd prefer to handle it in a separate patch.

V3 retains the fixes for completing `INDEX` after an explicitly typed
`USING` and looking up the correct table's indexes, including column-list
forms with an option list.

Thanks,
Sho

--
＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
*Sho Ito (Sean)*

E-mail: i(dot)sho(dot)0628(at)gmail(dot)com
LinkedIn: *https://www.linkedin.com/in/sho-ito-0628/
<https://www.linkedin.com/in/sho-ito-0628/>*
GitHub: https://github.com/sean0628/
＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝