[feat](jni-writer) Support JNI-based write framework for custom writers#60756
[feat](jni-writer) Support JNI-based write framework for custom writers#60756morningman merged 2 commits intoapache:masterfrom
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
|
PR approved by anyone and no changes requested. |
FE UT Coverage ReportIncrement line coverage |
| fileWriter.write(output); | ||
| writtenRows += numRows; | ||
| writtenBytes += output.getBytes().length; | ||
| LOG.info("LocalFileJniWriter wrote " + numRows + " rows, totalWrittenRows=" + writtenRows |
There was a problem hiding this comment.
L65 and L92
so many logs in write, maybe could remove next time
There was a problem hiding this comment.
OK. This is just a demo writer.
|
PR approved by at least one committer and no changes requested. |
TPC-H: Total hot run time: 29187 ms |
TPC-DS: Total hot run time: 183519 ms |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
What problem does this PR solve?
Related #60768
Introduce a symmetric JNI writer mechanism that complements the existing
JNI scanner framework, enabling custom file writers to be implemented in
Java and invoked through JNI from the C++ query executor.
Key changes:
The implementation follows the same patterns as JniScanner for consistency:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)