-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](hudi) upgrade hudi to 0.15.0 #44267
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
TPC-H: Total hot run time: 39897 ms |
TPC-DS: Total hot run time: 196370 ms |
ClickBench: Total hot run time: 33.32 s |
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40093 ms |
TPC-DS: Total hot run time: 197412 ms |
ClickBench: Total hot run time: 32.65 s |
|
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
|
clang-tidy review says "All clean, LGTM! 👍" |
|
run buildall |
1 similar comment
|
run buildall |
TPC-H: Total hot run time: 40408 ms |
|
TeamCity be ut coverage result: |
0141378 to
9147fd3
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
3 similar comments
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
081336b to
d0c8116
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
|
clang-tidy review says "All clean, LGTM! 👍" |
04a6609 to
35a4152
Compare
...extensions/hadoop-hudi-scanner/src/main/java/org/apache/doris/hudi/HadoopHudiJniScanner.java
Outdated
Show resolved
Hide resolved
|
run buildall |
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40078 ms |
TPC-DS: Total hot run time: 196867 ms |
ClickBench: Total hot run time: 33.71 s |
|
PR approved by at least one committer and no changes requested. |
### What problem does this PR solve? 1. upgrade hudi to 0.15.0. 2. impl new hudi jni reader based on hudi-hadoop-mr 3. add session variable `hudi_jni_scanner` to choose which hudi jni reader to use, "hadoop" means HadoopHudiJniReader, "spark" means old HudiJniReader, default value is "hadoop" 4. support session variable `force_jni_scanner` for hudi 5. add more cases for hudi p2 ### Release note [opt](hudi) upgrade hudi to 0.15 and support hadoop jni reader
1. upgrade hudi to 0.15.0. 2. impl new hudi jni reader based on hudi-hadoop-mr 3. add session variable `hudi_jni_scanner` to choose which hudi jni reader to use, "hadoop" means HadoopHudiJniReader, "spark" means old HudiJniReader, default value is "hadoop" 4. support session variable `force_jni_scanner` for hudi 5. add more cases for hudi p2 [opt](hudi) upgrade hudi to 0.15 and support hadoop jni reader
### What problem does this PR solve? Related PR: #44267 Problem Summary: If we set hudi_jni_scanner to an incorrect value, jni_connector will be null, causing a core dump. So we set hadoop as default hudi jni scanner.
### What problem does this PR solve? Related PR: #44267 Problem Summary: If we set hudi_jni_scanner to an incorrect value, jni_connector will be null, causing a core dump. So we set hadoop as default hudi jni scanner.
### What problem does this PR solve? Related PR: apache#44267 Problem Summary: If we set hudi_jni_scanner to an incorrect value, jni_connector will be null, causing a core dump. So we set hadoop as default hudi jni scanner.
What problem does this PR solve?
Problem Summary:
upgrade hudi to 0.15.0 to avoid potential class not found problems
Release note
hudi_jni_scannerto choose which hudi jni reader to use, "hadoop" means HadoopHudiJniReader, "spark" means old HudiJniReader, default value is "hadoop"force_jni_scannerfor hudiCheck List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)