Skip to content

Comments

arm compatibility#73

Closed
acookin wants to merge 1 commit intodatafusion-contrib:mainfrom
acookin:main
Closed

arm compatibility#73
acookin wants to merge 1 commit intodatafusion-contrib:mainfrom
acookin:main

Conversation

@acookin
Copy link

@acookin acookin commented Jan 14, 2025

Use c_char instead of i8 so this is not platform dependent.

@kou
Copy link
Member

kou commented Jan 14, 2025

Do you have a code that shows that the current code has this problem?
(How did you find this? Build time warning?)

@acookin
Copy link
Author

acookin commented Jan 29, 2025

Do you have a code that shows that the current code has this problem? (How did you find this? Build time warning?)

yes, it was a build time warning building on an arm-based host. Keeping this in draft for now, I will post repro instructions when I get a chance.

@kou
Copy link
Member

kou commented Jan 29, 2025

OK.

We can use arm based GitHub hosted runner: https://github.blog/news-insights/product-news/arm64-on-github-actions-powering-faster-more-efficient-build-systems/

Let's use it in our CI.

@acookin acookin closed this Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants