-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[fix](clone) Fix clone and alter tablet use same tablet path #34889 #36858
[fix](clone) Fix clone and alter tablet use same tablet path #34889 #36858
Conversation
run buildall |
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
TeamCity be ut coverage result: |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TeamCity be ut coverage result: |
…34889) The entire process is as follows: 1. Drop the tablet. 2. Successfully clone the tablet in full. 3. Start the incremental clone. 4. Start to move the tablet to the trash (the process of actually cleaning the data begins from step 1, where the tablet was dropped). 5. The incremental clone fails. 6. The incremental clone is successfully retried. Step 4 moved the data that was just pulled from the full clone to the trash, leading to data loss. The failure in step 5 of the incremental clone was also due to the deletion of the just-pulled snapshot data. Fix: When cloning, check the tablet status and determine if the tablet directory has already been moved to the trash directory. If it has not been moved to the trash, the clone thread should help move it to the trash directory.
2d5e1f1
to
67ab139
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TeamCity be ut coverage result: |
cherry pick from #34889