-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Add backup_log
system table
#53638
Add backup_log
system table
#53638
Conversation
# Conflicts: # src/Interpreters/Context.cpp # src/Interpreters/InterpreterSystemQuery.cpp
@vitlibar could you please take a look at this pull request? |
This is an automated comment for commit 132ab60 with description of existing statuses. It's updated for the latest CI running
|
Hello guys. I'm trying to figure out what's wrong with the CI jobs. Have they been cancelled manually or disrupted by some preceding failure? |
You can try adding an empty commit to re-trigger the build. |
Sure, thanks. I might as well merge master into my branch and thus re-launch the CI. |
Maybe it's some kind of a bug in CI. I hope someone from ClickHouse like @vitlibar will explain it. |
@vitlibar, could you please review this PR? Because @sirvickr is already working on the related task about So ideally it would be nice to merge the current PR to master at the end of this week. Then @sirvickr could create new PR about the system.backups persistence at the beginning or the middle of the next week. |
@vitlibar, what do you think about this pull request? If you have any concerns or ideas, please feel free to share. |
No, let's wait until the checks are done and I will merge it. |
@sirvickr a bug has been found automatically: https://s3.amazonaws.com/clickhouse-test-reports/35961/c9753d7fd99050f0b6d4c4c5a02ab2c265239879/stateless_tests__tsan__[3_5]/stderr.log |
Thank you, I'll take a close look. |
I think I found the origin of the bug. The data race is possible due to the assignment of the I'm going to double check everything locally and make a fixing PR ASAP. |
Thank you! Please do 🙏 |
Closes #53221
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Added a new system log table
backup_log
to track allBACKUP
andRESTORE
operationsDocumentation entry for user-facing changes