yihua commented on code in PR #13687:
URL: https://github.com/apache/hudi/pull/13687#discussion_r2267043670
##########
hudi-spark-datasource/hudi-spark/src/test/java/org/apache/hudi/functional/TestHoodieBackedMetadata.java:
##########
@@ -2657,6 +2657,7 @@ public void testCleaningArchivingAndCompaction() throws
Exception {
}
}
+ @Disabled
Review Comment:
We need to rewrite this test to cover the similar flow for MDT reinit
between table version 6 and 8.
##########
hudi-spark-datasource/hudi-spark/src/test/java/org/apache/hudi/functional/TestHoodieBackedMetadata.java:
##########
@@ -2729,6 +2730,7 @@ public void testUpgradeDowngrade() throws IOException {
* When table needs to be upgraded and when multi writer is enabled, hudi
rolls back partial commits. Upgrade itself is happening
* within a lock and hence rollback should not lock again.
*/
+ @Disabled
Review Comment:
HUDI-9700 to track
##########
hudi-spark-datasource/hudi-spark/src/test/java/org/apache/hudi/functional/TestHoodieBackedMetadata.java:
##########
@@ -2729,6 +2730,7 @@ public void testUpgradeDowngrade() throws IOException {
* When table needs to be upgraded and when multi writer is enabled, hudi
rolls back partial commits. Upgrade itself is happening
* within a lock and hence rollback should not lock again.
*/
+ @Disabled
Review Comment:
Same here
##########
hudi-spark-datasource/hudi-spark/src/test/scala/org/apache/spark/sql/hudi/procedure/TestUpgradeOrDowngradeProcedure.scala:
##########
@@ -33,7 +33,7 @@ import scala.collection.JavaConverters._
class TestUpgradeOrDowngradeProcedure extends HoodieSparkProcedureTestBase {
- test("Test Call downgrade_table and upgrade_table Procedure") {
+ ignore("Test Call downgrade_table and upgrade_table Procedure") {
withTempDir { tmp =>
Review Comment:
These need to be updated using newer table versions so that the upgrade and
downgrade procedure have test coverage.
--
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]
