rangareddy commented on issue #17358: URL: https://github.com/apache/hudi/issues/17358#issuecomment-5351238710
This issue was reviewed as part of the JIRA-migrated backlog triage (HUDI-8837). **Findings: needs the strict validation re-added and the test run; the committed suite passing does not settle it.** The test named in the description exists on `master` at `hudi-spark-datasource/hudi-spark/src/test/scala/org/apache/hudi/functional/TestDataSourceForBootstrap.scala:408` (`testMetadataBootstrapMORPartitionedInlineCompactionOn`). The important caveat: per the description, the strict data validation that exposes the null partition path was added *during investigation* and is not part of the committed test. So the fact that this suite currently passes in CI is not evidence that the bug is gone - the assertions that would catch it are not there. Commits touching that file since the report are all unrelated: `#17608` (`latestCommitCompletionTime` consolidation), `#14060` (glob path removal), `#13982` (exclusive start commit for incremental query), `#13718` (precombine config key), `#12596` (column stats enabled by default). To progress this, the strict assertions need re-adding and the test rerunning against current `master`. If the null partition path still appears, that validation is worth committing alongside the fix so the case cannot silently regress again. The JIRA is marked In Progress with an assignee, so there may be findings not reflected in this issue - worth syncing before anyone else starts. 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]
