Surely it isn't right to add members prefixed with "ioss_" to struct IndexScanState.
I'm surprised about this "FirstTupleEmitted" business. Wouldn't it make more sense to implement index_skip() to return the first tuple if the scan is just starting? (I know little about executor, apologies if this is a stupid question.) It would be good to get more knowledgeable people to review this patch. It's clearly something we want, yet it's been there for a very long time. Thanks -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
