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

Add last_exception_time to replication_queue #45457

Merged
merged 2 commits into from Feb 3, 2023

Conversation

FrankChen021
Copy link
Contributor

Changelog category (leave one):

  • New Feature

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Add a new column last_exception_time to system.replication_queue

Two changes are contained in this PR:

  1. a new column last_exception_time is added to replication_queue to reflect the time when the last exception is raised
    2.the full stack trace of the last exception is saved instead of the exception message

These changes save us a lot of time to search the text log to find out when and what the exact exception occurs.

@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-feature Pull request with new product feature label Jan 20, 2023
@tavplubix tavplubix added the can be tested Allows running workflows for external contributors label Jan 20, 2023
@tavplubix tavplubix self-assigned this Jan 20, 2023
@tavplubix tavplubix marked this pull request as draft January 30, 2023 13:40
@FrankChen021
Copy link
Contributor Author

Hello @tavplubix , may I know why is it marked as draft?

@tavplubix
Copy link
Member

Because of failed tests

@FrankChen021
Copy link
Contributor Author

Noted, let me check

@FrankChen021 FrankChen021 marked this pull request as ready for review February 3, 2023 06:55
@tavplubix
Copy link
Member

Integration tests (tsan) [6/6] - test_azure_blob_storage_zero_copy_replication is known to be flaky
Stateless tests (release, DatabaseReplicated) [3/4] - 02540_date_column_consistent_insert_behaviour was broken in master

@tavplubix tavplubix merged commit 7e6f7c7 into ClickHouse:master Feb 3, 2023
@FrankChen021 FrankChen021 deleted the exception_time branch February 6, 2023 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
can be tested Allows running workflows for external contributors pr-feature Pull request with new product feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants