On 2020/01/29 20:06, Kasahara Tatsuhito wrote:
> Hi.
>
> Attached patch solve this problem.
>
> This patch adds table_beginscan_tid() and call it in TidListEval()
> instead of table_beginscan().
> table_beginscan_tid() is the same as table_beginscan() but do not set
> SO_TYPE_SEQSCAN to flags.
>
> Although I'm not sure this behavior is really problem or not,
> it seems to me that previous behavior is more prefer.
>
> Is it worth to apply to HEAD and v12 branch ?
I've not read the patch yet, but I agree that updating only seq_scan
but not seq_tup_read in Tid Scan sounds strange. IMO at least
both should be update together or neither should be updated.
Regards,
--
Fujii Masao
NTT DATA CORPORATION
Advanced Platform Technology Group
Research and Development Headquarters