Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Improvement]: Optimize can't stop when the change store only has eq-delete files and the base store has no files #814

Closed
3 tasks done
hzluting opened this issue Dec 2, 2022 · 3 comments · Fixed by #815

Comments

@hzluting
Copy link
Contributor

hzluting commented Dec 2, 2022

Search before asking

  • I have searched in the issues and found no similar issues.

What would you like to be improved?

Even if the minor optimize output is nothing, the commit still should update "max-txId" in partition properties.

How should we improve?

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Subtasks

No response

Code of Conduct

hzluting pushed a commit to hzluting/arctic that referenced this issue Dec 2, 2022
Even if the minor optimize output is nothing, the commit still update "max-txId" in partition properties.
hzluting pushed a commit to hzluting/arctic that referenced this issue Dec 2, 2022
add unit test
hzluting pushed a commit to hzluting/arctic that referenced this issue Dec 2, 2022
fix unit test
@zhoujinsong
Copy link
Contributor

zhoujinsong commented Dec 2, 2022

It seems this is a complicated situation.
Could you give more detail to help us understand it? like:

  • Table types that the problem may affect
  • What is the situation the problem occurs
  • What do you think causes the problem and how should we fix it

@hzluting
Copy link
Contributor Author

hzluting commented Dec 2, 2022

It seems this is a complicated situation. Could you give more detail to help us understand it? like:

  • Table types that the problem may affect
  • What is the situation the problem occurs
  • What do you think causes the problem and how should we fix it

The problem only occurs in the arctic keyed table. There are no files in the base store and only some eq-delete files in the change store. So the table will trigger minor optimize, but this optimize will output nothing. Then optimize commit will skip, because there are no changes for the base store, so "max-txid" in the base store property won't be updated, and the minor optimize won't stop.
I will add logic to update "max-txid" in the base store property, even if minor optimize doesn't produce any files.

@zhoujinsong
Copy link
Contributor

It seems this is a complicated situation. Could you give more detail to help us understand it? like:

  • Table types that the problem may affect
  • What is the situation the problem occurs
  • What do you think causes the problem and how should we fix it

The problem only occurs in the arctic keyed table. There are no files in the base store and only some eq-delete files in the change store. So the table will trigger minor optimize, but this optimize will output nothing. Then optimize commit will skip, because there are no changes for the base store, so "max-txid" in the base store property won't be updated, and the minor optimize won't stop. I will add logic to update "max-txid" in the base store property, even if minor optimize doesn't produce any files.

Get it. Thanks for your reply.

hzluting pushed a commit to hzluting/arctic that referenced this issue Dec 2, 2022
fix code review
zhoujinsong pushed a commit that referenced this issue Dec 2, 2022
…q-delete files and the base store has no files (#815)

* fix #814
Even if the minor optimize output is nothing, the commit still update "max-txId" in partition properties.

* fix #814
add unit test

* fix #814
fix unit test

* fix #814
fix code review

Co-authored-by: luting <dylzlt93299@gmail.com>
wangtaohz pushed a commit that referenced this issue Dec 7, 2022
…q-delete files and the base store has no files (#815)

* fix #814
Even if the minor optimize output is nothing, the commit still update "max-txId" in partition properties.

* fix #814
add unit test

* fix #814
fix unit test

* fix #814
fix code review

Co-authored-by: luting <dylzlt93299@gmail.com>
zhoujinsong pushed a commit that referenced this issue May 31, 2023
…q-delete files and the base store has no files (#815)

* fix #814
Even if the minor optimize output is nothing, the commit still update "max-txId" in partition properties.

* fix #814
add unit test

* fix #814
fix unit test

* fix #814
fix code review

Co-authored-by: luting <dylzlt93299@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants