Aggarwal-Raghav commented on PR #6880:
URL: https://github.com/apache/hive/pull/6880#issuecomment-6200106709
```
====================================================================================================
TEST RESULTS
====================================================================================================
1. TestInputFormatReaderDeletes
- Before change: Tests run: 56, Failures: 0, Errors: 0 (Time: 20.42 s)
- After change: Tests run: 88, Failures: 0, Errors: 0, Skipped: 2
(Time: 34.25 s)
- Status: PASS (+32 test executions gained; tested across
Parquet/Avro/ORC, V2/V3, and Mapred/MR)
2. TestIcebergInputFormats
- Before change: Tests run: 72, Failures: 0, Errors: 0, Skipped: 6
(Time: 10.72 s)
- After change: Tests run: 68, Failures: 0, Errors: 0, Skipped: 1
(Time: 11.29 s)
- Status: PASS (migrated to JUnit 5 parameterized; validates upstream
FileHelpers, GenericAppenderHelper, RandomGenericData)
- Reason for count difference:
• Executed tests increased (66 -> 67): added new upstream test
(+6) and stopped running the format-independent LLAP test 6 redundant times
(-5).
• Skipped tests decreased (6 -> 1): JUnit 5 counts disabled tests
once instead of per parameter tuple.
3. TestCatalogs
- Before change: Tests run: 11, Failures: 0, Errors: 0 (Time: 0.73 s)
- After change: Tests run: 11, Failures: 0, Errors: 0, Skipped: 0
(Time: 0.77 s)
- Status: PASS (migrated to JUnit 5 & AssertJ; validates custom
catalog property tests)
4. TestHiveIcebergDeleteWriter
- Before change: Tests run: 5, Failures: 0, Errors: 0 (Time: 3.65 s)
- After change: Tests run: 5, Failures: 0, Errors: 0, Skipped: 0
(Time: 3.60 s)
- Status: PASS (validates HiveIcebergWriterTestBase &
TestHelper.toPath)
5. TestHiveVectorizedReader
- Before change: Tests run: 1, Failures: 0, Errors: 0 (Time: 1.70 s)
- After change: Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
(Time: 1.71 s)
- Status: PASS (validates TestHelper.toPath backward compatibility)
6. Removed Duplicate Utilities (TestCharSequenceSet, TestLocationUtil)
- Before change: Tests run: 3, Failures: 0, Errors: 0 (Time: 0.06 s)
- After change: Removed duplicate classes; verified covered upstream
via iceberg-core:tests jar
- Status: PASS
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
