apache / cloudberry

One advanced and mature open-source MPP (Massively Parallel Processing) database. Open source alternative to Greenplum Database.
https://cloudberry.apache.org
Apache License 2.0
430 stars 104 forks source link

Fix reference leak on AO/AOCS partition tables with unique index. #649

Closed avamingli closed 1 month ago

avamingli commented 1 month ago

Fix https://github.com/cloudberrydb/cloudberrydb/issues/557.

This issue also exists for AOCS.

This is introduced by GPDB commit ed364586b99. For AO/AOCS partition tables, it's possible to update across partitions. And it does have deleteDesc and uniqueCheckDesc if there were. The DeleteDesc and InsertDesc are not enough to decide if the partition has a update operation or not. Some partitions have visimap but some not, clean them if possible.

Authored-by: Zhang Mingli avamingli@gmail.com

fix #ISSUE_Number


Change logs

Describe your change clearly, including what problem is being solved or what feature is being added.

If it has some breaking backward or forward compatibility, please clary.

Why are the changes needed?

Describe why the changes are necessary.

Does this PR introduce any user-facing change?

If yes, please clarify the previous behavior and the change this PR proposes.

How was this patch tested?

Please detail how the changes were tested, including manual tests and any relevant unit or integration tests.

Contributor's Checklist

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

avamingli commented 1 month ago

Pushed, thanks for review.