Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/23613 )
Change subject: IMPALA-14092 Part2: Support querying of paimon data table via JNI ...................................................................... Patch Set 14: Code-Review+1 (4 comments) I think we should plan to integrate Paimon scanner with Impala io-manager for read fairness. I'm OK to punt this to next iteration. http://gerrit.cloudera.org:8080/#/c/23613/14/be/src/exec/paimon/CMakeLists.txt File be/src/exec/paimon/CMakeLists.txt: http://gerrit.cloudera.org:8080/#/c/23613/14/be/src/exec/paimon/CMakeLists.txt@53 PS14, Line 53: # NOTE: the fix below only applies to current version of toolchain. : # need to adapt if toolchain is upgraded. Filed IMPALA-14607 as follow up. http://gerrit.cloudera.org:8080/#/c/23613/12/be/src/exec/paimon/paimon-jni-scan-node.cc File be/src/exec/paimon/paimon-jni-scan-node.cc: http://gerrit.cloudera.org:8080/#/c/23613/12/be/src/exec/paimon/paimon-jni-scan-node.cc@143 PS12, Line 143: env, &array, &schema, &row_ > clang-tidy complain here. Done http://gerrit.cloudera.org:8080/#/c/23613/12/fe/src/main/java/org/apache/impala/catalog/paimon/PaimonUtil.java File fe/src/main/java/org/apache/impala/catalog/paimon/PaimonUtil.java: http://gerrit.cloudera.org:8080/#/c/23613/12/fe/src/main/java/org/apache/impala/catalog/paimon/PaimonUtil.java@879 PS12, Line 879: pub > nit: add a whitespace before. Done http://gerrit.cloudera.org:8080/#/c/23613/13/fe/src/main/java/org/apache/impala/planner/paimon/PaimonSplit.java File fe/src/main/java/org/apache/impala/planner/paimon/PaimonSplit.java: http://gerrit.cloudera.org:8080/#/c/23613/13/fe/src/main/java/org/apache/impala/planner/paimon/PaimonSplit.java@28 PS13, Line 28: /** : * Paimon split entity used by paimon jni scanner. : * */ : public class PaimonSplit implements Serializable Done -- To view, visit http://gerrit.cloudera.org:8080/23613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie679a89a8cc21d52b583422336b9f747bdf37384 Gerrit-Change-Number: 23613 Gerrit-PatchSet: 14 Gerrit-Owner: ji chen <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: ji chen <[email protected]> Gerrit-Comment-Date: Fri, 05 Dec 2025 18:12:25 +0000 Gerrit-HasComments: Yes
