kirkrodrigues commented on PR #13687: URL: https://github.com/apache/pinot/pull/13687#issuecomment-2264367874
> After https://github.com/apache/pinot/pull/13648, these tests should be now skipped by default when being executed on aarch64. Have you verify they have being executed? Yep, my branch was implemented before #13648 was merged and I had tested that the tests ran on an Apple Silicon Mac. I tested again just now on a Linux aarch64 machine (by removing the `TestUtils.ensureArchitectureIsNotARM()` check and running the CLP tests using clp-ffi@0.4.5). Should we remove the `TestUtils.ensureArchitectureIsNotARM()` check in this PR as well? -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
