prashantwason commented on PR #18083: URL: https://github.com/apache/hudi/pull/18083#issuecomment-4029988857
Pushed a fix for the CI test failures: **Fixed 5 validation tests** (`testInvalidDrainSizeNegative`, `testInvalidDrainSizeZero`, `testInvalidBufferSizeZero`, `testInvalidBufferSizeNegative`, `testInvalidSortKeysOnlyWhitespace`, `testInvalidSortKeysOnlyCommas`): - Moved config validation before `super.open()` which requires a Flink runtime context. Tests that directly instantiate the function without a test harness were getting `IllegalStateException` instead of `IllegalArgumentException`. **Fixed 2 sort order tests** (`testSortedResult`, `testSortedResultWithContinuousDrain`): - Added `RecordComparator` for full comparison fallback when normalized keys are equal. Normalized keys are a prefix optimization and may not fully encode all sort fields, so a full `RecordComparator` is needed for correct multi-field sort order. The Azure CI failure was infrastructure-related (Docker process exit code 1), not a test failure. -- 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]
