[GitHub] [arrow] rongma1997 commented on pull request #7326: ARROW-9010: [Java] Framework and interface changes for RecordBatch IPC buffer compression

2020-06-15 Thread GitBox
rongma1997 commented on pull request #7326: URL: https://github.com/apache/arrow/pull/7326#issuecomment-644536052 Hi, I have a question... probably not related to what this PR focus on. What if the compressor / decompressor for the codec will have JNI call for compression / decomperssio

[GitHub] [arrow] emkornfield commented on pull request #7231: ARROW-6839: [Java] Add APIs to read and write "custom_metadata" field of IPC file footer

2020-06-15 Thread GitBox
emkornfield commented on pull request #7231: URL: https://github.com/apache/arrow/pull/7231#issuecomment-644543412 No please go ahead and merge. This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [arrow] kiszk commented on pull request #6729: ARROW-8229: [Java] Move ArrowBuf into the Arrow package

2020-06-15 Thread GitBox
kiszk commented on pull request #6729: URL: https://github.com/apache/arrow/pull/6729#issuecomment-644546508 I see. Since `ArrowBuf` package name is changed, IMHO, a PR for this change in Spark should include upgrading Arrow from 0.15.1 to 0.17.1. Since Spark 3.0 branch is already cu

[GitHub] [arrow] tianchen92 merged pull request #7231: ARROW-6839: [Java] Add APIs to read and write "custom_metadata" field of IPC file footer

2020-06-15 Thread GitBox
tianchen92 merged pull request #7231: URL: https://github.com/apache/arrow/pull/7231 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

[GitHub] [arrow] emkornfield commented on pull request #7231: ARROW-6839: [Java] Add APIs to read and write "custom_metadata" field of IPC file footer

2020-06-15 Thread GitBox
emkornfield commented on pull request #7231: URL: https://github.com/apache/arrow/pull/7231#issuecomment-644556734 @tianchen92 did you use the merge tool to merge this (if you haven't please read https://cwiki.apache.org/confluence/display/ARROW/Guide+for+Committers+and+Project+Maintainers

[GitHub] [arrow] tianchen92 commented on pull request #7231: ARROW-6839: [Java] Add APIs to read and write "custom_metadata" field of IPC file footer

2020-06-15 Thread GitBox
tianchen92 commented on pull request #7231: URL: https://github.com/apache/arrow/pull/7231#issuecomment-644557502 > @tianchen92 did you use the merge tool to merge this (if you haven't please read https://cwiki.apache.org/confluence/display/ARROW/Guide+for+Committers+and+Project+Maintainer

[GitHub] [arrow] liyafan82 commented on pull request #6729: ARROW-8229: [Java] Move ArrowBuf into the Arrow package

2020-06-15 Thread GitBox
liyafan82 commented on pull request #6729: URL: https://github.com/apache/arrow/pull/6729#issuecomment-644559038 > @liyafan82 OK. Could you open an JIRA issue for Spark to notify this to Spark developers? https://issues.apache.org/jira/browse/SPARK > FYI: https://spark.apache.org/contrib

[GitHub] [arrow] liyafan82 commented on pull request #7326: ARROW-9010: [Java] Framework and interface changes for RecordBatch IPC buffer compression

2020-06-15 Thread GitBox
liyafan82 commented on pull request #7326: URL: https://github.com/apache/arrow/pull/7326#issuecomment-644563536 > Hi, I have a question... probably not related to what this PR focus on. > What if the compressor / decompressor for the codec will have JNI call for compression / decomperss

[GitHub] [arrow] emkornfield commented on pull request #7231: ARROW-6839: [Java] Add APIs to read and write "custom_metadata" field of IPC file footer

2020-06-15 Thread GitBox
emkornfield commented on pull request #7231: URL: https://github.com/apache/arrow/pull/7231#issuecomment-644564824 Not familiar enough with the implications @kszucs @kou what are the correct steps here? This is an automated

<    1   2