Zhenghua Lyu
Zhenghua Lyu
> I have run the full pipeline in my local, and all tests of the resource group are passed. Thus we can push this directly since it only modifies test...
Hi @ashwinstar @vraghavan78 > If it is unimportant which optimizer is used, why change it? a. some cases have different output (mem usage) under different optimizer b. enable cassert or...
Super job!!!!! From my side: 1. we need more commments 2. we need to add test cases 3. some code refinement still need. But the test is super! Thanks a...
I have rebased this PR with latest 6X and run a full local pipeline, it is green. I'll push this now. Thanks a lot for this contribution!
Hi, interesting PR, any plan to make progress?
Short version: * for main branch: remove `simple_equality_predicate_refuted` and all related code * for 6x: do nothing (until later we hear real complains). the close this issue. ------------ TL; DR...
Fixed by https://github.com/greenplum-db/gpdb/pull/14553 in main branch. Close it. (for 6x do nothing until we hear real complains). @zxuejing I recommend to add the case in this issue to main branch...
Close this since this is fixed by https://github.com/greenplum-db/gpdb/pull/14553 in main branch. For 6X we don't want to change behavior until we hear from real complains.
I think we can just close this issue. For me doc change is not needed.
@zxuejing @huansong will this https://github.com/greenplum-db/gpdb/commit/578bd4236880f58272b3a291f5cb659208e3397a fix the issue?