apache / orc

Apache ORC - the smallest, fastest columnar storage for Hadoop workloads
https://orc.apache.org/
Apache License 2.0
689 stars 483 forks source link

ORC-1541: Add `Ubuntu 24.04 LTS` Docker Test #1679

Closed dongjoon-hyun closed 11 months ago

dongjoon-hyun commented 11 months ago

What changes were proposed in this pull request?

This PR aims to add Ubuntu 24.04 LTS Docker Test.

Why are the changes needed?

Although the official Ubuntu 24.04 LTS will arrive on April 2024, we can start to track it from the official docker dev image.

How was this patch tested?

Manual review.

$ ./reinit.sh ubuntu24
$ ./run-one.sh local ORC-1541 ubuntu24
...
Test project /root/build
    Start 1: orc-test
1/8 Test #1: orc-test .........................   Passed    4.53 sec
    Start 2: java-test
2/8 Test #2: java-test ........................   Passed  104.55 sec
    Start 3: java-tools-test
3/8 Test #3: java-tools-test ..................   Passed    0.07 sec
    Start 4: java-bench-gen-test
4/8 Test #4: java-bench-gen-test ..............   Passed    0.74 sec
    Start 5: java-bench-scan-test
5/8 Test #5: java-bench-scan-test .............   Passed    0.69 sec
    Start 6: java-bench-hive-test
6/8 Test #6: java-bench-hive-test .............   Passed   11.21 sec
    Start 7: java-bench-spark-test
7/8 Test #7: java-bench-spark-test ............   Passed    3.36 sec
    Start 8: tool-test
8/8 Test #8: tool-test ........................   Passed    6.94 sec

100% tests passed, 0 tests failed out of 8

Total Test time (real) = 132.08 sec
Built target test-out
Finished ubuntu24 at Fri Dec  8 23:34:01 PST 2023

This closes #1501 .

dongjoon-hyun commented 11 months ago

This is currently blocked by libhdfs failure. I'm looking at this.

/root/build/libhdfspp_ep-prefix/src/libhdfspp_ep/include/hdfspp/uri.h:60:3: error: 'uint16_t' does not name a type
   60 |   uint16_t get_port() const;
dongjoon-hyun commented 11 months ago

Currently, it's blocked by the following.

[ RUN      ] TestTimestampStatistics.testOldFile
unknown file: Failure
C++ exception with description "Time zone file /usr/share/zoneinfo/US/Pacific does not exist. Please install IANA time zone database and set TZDIR env." thrown in the test body.
dongjoon-hyun commented 11 months ago

I manually verified manually.