Skip to content

Conversation

@yihua
Copy link
Contributor

@yihua yihua commented May 6, 2023

Change Logs

The test TestIncrementalReadWithFullTableScan#testFailEarlyForIncrViewQueryForNonExistingFiles can fail due to changes in archival behavior because of hard-coded parameters. This PR improves the test to be more robust.

Impact

As above.

Risk level

none

Documentation Update

N/A

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@zhangyue19921010
Copy link
Contributor

@hudi-bot run azure

@hudi-bot
Copy link
Collaborator

hudi-bot commented May 6, 2023

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@yihua
Copy link
Contributor Author

yihua commented May 6, 2023

The CI failure is due to other failed tests. Merging this PR.

@yihua yihua merged commit 3dcd757 into apache:master May 6, 2023
yihua added a commit to yihua/hudi that referenced this pull request May 15, 2023
The test TestIncrementalReadWithFullTableScan#testFailEarlyForIncrViewQueryForNonExistingFiles can fail due to changes in archival behavior because of hard-coded parameters. This commit improves the test to be more robust.
yihua added a commit to yihua/hudi that referenced this pull request May 15, 2023
The test TestIncrementalReadWithFullTableScan#testFailEarlyForIncrViewQueryForNonExistingFiles can fail due to changes in archival behavior because of hard-coded parameters. This commit improves the test to be more robust.
yihua added a commit to yihua/hudi that referenced this pull request May 17, 2023
The test TestIncrementalReadWithFullTableScan#testFailEarlyForIncrViewQueryForNonExistingFiles can fail due to changes in archival behavior because of hard-coded parameters. This commit improves the test to be more robust.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants