[AMORO-3218] Fix logic for consturct the result of optimizer tables api

apache / amoro

Apache Amoro (incubating) is a Lakehouse management system built on open data lake formats.

https://amoro.apache.org/

Apache License 2.0

878 stars 293 forks source link

[AMORO-3218] Fix logic for consturct the result of optimizer tables api #3231

Closed klion26 closed 1 month ago

klion26 commented 2 months ago

Why are the changes needed?

Close #3218 .

Brief change log

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
[ ] Add screenshots for manual tests if appropriate
[x] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

klion26 commented 2 months ago

cc @huyuanfeng2018 @zhoujinsong @majin1102

klion26 commented 1 month ago

@majin1102 I've addressed the comments, please take another look when you're free. thanks.

klion26 commented 1 month ago

@huyuanfeng2018 @majin1102 thanks for the review and merging!

zhoujinsong commented 1 month ago

@klion26

I cannot cherrypick this PR into branch 0.7.x due to some conflicts. Can you help to open another PR to branch 0.7.x to help us fix this issue in branch 0.7.x?