Re: [PR] Add Arrow C Data Interface and nanoarrow dependency [iceberg-cpp]

2025-03-22 Thread via GitHub
Fokko commented on PR #44: URL: https://github.com/apache/iceberg-cpp/pull/44#issuecomment-2742715984 This looks good to me, thanks @wgtmac for working on this, and thanks @lidavidm and @zhjwpku for the review 👍 -- This is an automated message from the Apache Git Service. To respond to t

Re: [PR] Add Arrow C Data Interface and nanoarrow dependency [iceberg-cpp]

2025-03-21 Thread via GitHub
Fokko merged PR #44: URL: https://github.com/apache/iceberg-cpp/pull/44 -- 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 the specific comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apach

Re: [PR] Add Arrow C Data Interface and nanoarrow dependency [iceberg-cpp]

2025-03-21 Thread via GitHub
wgtmac commented on PR #44: URL: https://github.com/apache/iceberg-cpp/pull/44#issuecomment-2742728263 Thank you, @Fokko! -- 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 the specific comment. To

Re: [PR] Add Arrow C Data Interface and nanoarrow dependency [iceberg-cpp]

2025-03-19 Thread via GitHub
wgtmac commented on PR #44: URL: https://github.com/apache/iceberg-cpp/pull/44#issuecomment-2738966373 @Xuanwo @Fokko Could you help review this? -- 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 t

Re: [PR] Add Arrow C Data Interface and nanoarrow dependency [iceberg-cpp]

2025-03-19 Thread via GitHub
zhjwpku commented on PR #44: URL: https://github.com/apache/iceberg-cpp/pull/44#issuecomment-2738871141 Agreed, nanoarrow sounds good to me. -- 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 the sp