rangareddy commented on issue #17221:
URL: https://github.com/apache/hudi/issues/17221#issuecomment-5366109813
This issue was reviewed as part of the JIRA-migrated backlog triage
(HUDI-3668).
**Findings: still open, and the module is covered by several overlapping
tickets.**
The body carries a large amount of failure output naming specific cases -
for example
`org.apache.hudi.integ.testsuite.TestDFSHoodieTestSuiteWriterAdapter#testDFSTwoFilesWriteWithRollover`
failing with Mockito `TooManyActualInvocations`
(`avroFileDeltaInputWriter.canWrite()` wanted 2 times, was 3), which is a
test-expectation drift rather than a product bug.
The wider problem is ownership. `hudi-integ-test` is currently the subject
of at least three open tickets with no single owner of its health:
- this one - failing unit tests in the module
- #17276 (HUDI-8274) - `ITTestHoodieSanity` and `ITTestHoodieSyncCommand`
fully `@Disabled("HUDI-8274")`, with the description itself suggesting they may
be deletable
- #17240 (HUDI-6904) - flaky integration tests
Given #17276 raises the possibility that some of this code should simply be
removed, it is worth deciding the module's future first. Fixing test
expectations in code that is a candidate for deletion is wasted effort, and the
reverse - deleting tests that were about to be fixed - loses coverage.
Keeping this open.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
