nsivabalan commented on issue #16887: URL: https://github.com/apache/hudi/issues/16887#issuecomment-4800669857
Looks like the premise here may be stale. On current `master`, both handlers have dedicated unit-test coverage: - `hudi-client/hudi-client-common/src/test/java/org/apache/hudi/table/upgrade/TestEightToSevenDowngradeHandler.java` — 4 test methods (incl. parameterized) covering `deleteMetadataPartition`, `downgradeMetadataPartitions`, `downgradePartitionFields`, `unsetInitialVersion`, `unsetRecordMergeMode`, `downgradeBootstrapIndexType`, `downgradeKeyGeneratorType`. - `hudi-client/hudi-client-common/src/test/java/org/apache/hudi/table/upgrade/TestSevenToEightUpgradeHandler.java` — covers `upgradePartitionFields`, `setInitialVersion`, `upgradeMergeMode` (parameterized over payload class / preCombine combos), `upgradeBootstrapIndexType`, `upgradeKeyGeneratorType`. - `hudi-spark-datasource/hudi-spark/src/test/scala/org/apache/hudi/functional/TestSevenToEightUpgrade.scala` — ~400 lines of functional/E2E coverage. These were added in #12327 (HUDI-7665) and expanded in #13046 (HUDI-8409). If there are specific scenarios still missing (e.g., MOR vs CoW matrix, specific metadata partition combinations, downgrade then upgrade round-trip), it would help to enumerate those concretely so the issue can be scoped — otherwise this could probably be closed. -- 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]
