[test](schema-change) log alter job state in test_schema_change_mow_with_empty_rowset#64903
Open
shuke987 wants to merge 1 commit into
Open
[test](schema-change) log alter job state in test_schema_change_mow_with_empty_rowset#64903shuke987 wants to merge 1 commit into
shuke987 wants to merge 1 commit into
Conversation
…ith_empty_rowset Add a logger.info for the alter table job state polled inside the Awaitility loop so failures are easier to diagnose from regression logs. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed changes
Add a
logger.infoto print the alter table job state that is polled inside the Awaitility loop intest_schema_change_mow_with_empty_rowset.groovy.The test waits for the schema-change job to reach
FINISHED/CANCELLEDbut does not log the intermediate state. When the job stalls or ends upCANCELLED, the regression log shows no trace of the observed state, making failures hard to diagnose. Logging the polledreseach iteration makes such failures self-explanatory from the test output.No behavior change to the test logic itself.
Checklist
🤖 Generated with Claude Code