Cherry-pick ORCA InPlaceUpdate implementation

foreyes commented 2 months ago

Cherry-pick some ORCA patches from GreenPlum. Only ORCA SplitUpdate and the submissions it depends on are selected.

Change logs

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Contributor's Checklist

Here are some reminders and checklists before/when submitting your pull request, please check them:

[ ] Make sure your Pull Request has a clear title and commit message. You can take git-commit template as a reference.
[ ] Sign the Contributor License Agreement as prompted for your first-time contribution(One-time setup).
[ ] Learn the coding contribution guide, including our code conventions, workflow and more.
[ ] List your communication in the GitHub Issues or Discussions (if has or needed).
[ ] Document changes.
[ ] Add tests for the change
[ ] Pass make installcheck
[ ] Pass make -C src/test installcheck-cbdb-parallel
[ ] Feel free to request cloudberrydb/dev team for review and approval when your PR is ready🥳

CLAassistant commented 2 months ago

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 7 committers have signed the CLA.

:white_check_mark: foreyes
:x: l-wang
:x: THANATOSLAVA
:x: khannaekta
:x: dgkimura
:x: Sanath97
:x: pobbatihari
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

jiaqizho commented 1 month ago

@foreyes address the conflicts and make CI pass pls.

without conflicts address, the CI won't running.

my-ship-it commented 1 month ago

@foreyes address the conflicts and make CI pass pls.

without conflicts address, the CI won't running.

@jiaqizho CI already passed, please continue to review.

jiaqizho commented 3 weeks ago

@foreyes any updated?

foreyes commented 3 weeks ago

@jiaqizho @leborchuk This MR was not updated because it caused some testing failures in our internal repository, and I am still trying to find out the reason behind it. But now there is no problem merging into the GitHub repository, and this MR only cherry picked some of the GreenPlum modifications and resolved conflicts. Maybe I can figure out the internal repo CI failure later.

Only ORCA SplitUpdate and the submissions it depends on are selected.

foreyes commented 3 weeks ago

I'll resolve the conflict and update this MR soon.

foreyes commented 3 weeks ago

@jiaqizho commit message updated and CI passed.

apache / cloudberry