Skip to content

[HUDI-6521] Disable failing test case.#9169

Merged
nsivabalan merged 4 commits intoapache:masterfrom
amrishlal:disable-failing-testcase
Jul 11, 2023
Merged

[HUDI-6521] Disable failing test case.#9169
nsivabalan merged 4 commits intoapache:masterfrom
amrishlal:disable-failing-testcase

Conversation

@amrishlal
Copy link
Contributor

Change Logs

The test case Test Bulk Insert Into Bucket Index Table in TestInsertTable.scala is failing as a result of changes made in ##9156. Commenting out the test case to unblock to the build.

Impact

None

Risk level (write none, low medium or high below)

None

Documentation Update

None

Contributor's checklist

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

}
}
}
// FAILING TEST CASE. DO NOT UNCOMMENT BEFORE FIXING THE TEST CASE (see https://github.com/apache/hudi/pull/9156)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is now how we disable tests. we use "Disabled" tag to disable tests. we need to add a hudi jira adding links to failures and stacktraces may be and then add jira no in the Disabled annotation. you can find examples in our test suite.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed using following syntax/structure

  /** Ignore failing test case (see HUDI-6521 for more details) */
  ignore("Test Bulk Insert Into Bucket Index Table") {
    ...
  }

@amrishlal amrishlal changed the title [HUDI-6515] Commenting out failing test case. [HUDI-6521] Commenting out failing test case. Jul 11, 2023
@amrishlal amrishlal changed the title [HUDI-6521] Commenting out failing test case. [HUDI-6521] Disable failing test case. Jul 11, 2023
@hudi-bot
Copy link
Collaborator

CI report:

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

@amrishlal
Copy link
Contributor Author

@hudi-bot run azure

@nsivabalan nsivabalan merged commit 51ddf1a into apache:master Jul 11, 2023
@stream2000
Copy link
Contributor

Hi @amrishlal and @nsivabalan, sorry for the test case failure introduced. I have raised #9163 and fixed ci. Should we just revert this pr and enable the test again? Thanks~

stream2000 added a commit to stream2000/hudi that referenced this pull request Jul 13, 2023
leesf pushed a commit that referenced this pull request Jul 13, 2023
@amrishlal amrishlal deleted the disable-failing-testcase branch August 7, 2023 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants