Sami I

Results 6 comments of Sami I

Thanks for the report @michaelpq After we spoke offline, I spent time looking further into this. What I observed is the follows: 1. skipping over setup_scan_method_enforcement https://github.com/ossc-db/pg_hint_plan/blob/master/pg_hint_plan.c#L3955-L3956 does not prevent...

> For now, I would like to choose a revert of https://github.com/ossc-db/pg_hint_plan/commit/684986acc3b3156aaf22c1571cd44dd34059a346 & friends on stable branches, getting back to the previous-still-somewhat-broken behavior. With the above said, I do agree...

I agree. I'll submit a patch for Parallel Hint tests ( including those mentioned above ).

Attached are tests that cover several of the gaps discovered in this discussion: 1/ Ensure that Parallel enforcement does not break when a non-existent index is used in an IndexScan...

>The patch you are proposing for the additional tests has no explanation about what's being done, I hesitated doing that as it's not the norm to add much documentation to...

After second thought, showing that the non-existing index does not prevent parallel hint enforcement is not necessary. So, the test now shows the important part. For empty tables, parallel hints...