Zach Musgrave

This is a defect. Indexes that aren't simple sets of columns are not well tested in the engine, and there are surely lots of places that complex index expressions don't...

We'll roll this into https://github.com/dolthub/dolt/issues/2945, please track there.

@VinaiRachakonda does this work now?

This is difficult to get exactly right, and comes down to differences in the YACC parser we use vs. the one MySQL uses. The MySQL docs are not entirely accurate...

We have made a lot of strides here, enough that I'm going to close this out. If you encounter a keyword difference in parsing that's impacting your use case, please...

I'm in favor of this. IndexLookup shouldn't have `Str`, it should still implement fmt.Stringer `IndexedTable` should either embed `sql.Table`, or else just become `IndexAddressable` `AccessIndex` should be `IndexedRowIter`

Hi friends at src-d, Can we get some eyeballs on these outstanding pull requests? We have a bunch of additional changes we've been building on top of these (including work...

Any chance we can get this reviewed soon? We are continuing to stack changes on top of this one, and it's going to be really challenging to merge them back...

I think this is a bug. Can you submit a patch?

Enough time has passed that I'm not certain that this bug still applies, and we definitely do not implement the Driver* interfaces in dolt, where indexed lookups work. Please reopen...