u70b3 commented on PR #159: URL: https://github.com/apache/paimon-cpp/pull/159#issuecomment-5104451630
> Thanks a lot for this patch, and for the detailed validation you did on AArch64. It is worth noting that, @SteNicholas is already driving AArch64 support, tracked in [alibaba/paimon-cpp#132](https://github.com/alibaba/paimon-cpp/issues/132). His upcoming PR is expected to bring complete AArch64 build support, which will also cover the char signedness issues you fixed here. Once his PR is ready, we'll ping you for review. As additional context: The Apache repository is still in the middle of the code migration, so we prioritize accepting new PRs in the [alibaba/paimon-cpp](https://github.com/alibaba/paimon-cpp) repository. Thanks again for your contribution! Thanks for the context. I wasn’t aware of the ongoing repository migration or the existing AArch64 work. I ran into this issue in my own AArch64 environment and fixed it along the way, so I opened this PR in case the changes and validation results might be useful. I’m happy to review the upcoming PR and validate it directly in my AArch64 environment once it’s ready. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
