snowflakedb / gosnowflake

Go Snowflake Driver
Apache License 2.0
292 stars 122 forks source link

update apache arrow to v16 #1102

Open ChronosMasterOfAllTime opened 4 months ago

ChronosMasterOfAllTime commented 4 months ago

Description

Resolves #1101

Checklist

github-actions[bot] commented 4 months ago

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

ChronosMasterOfAllTime commented 4 months ago

I have read the CLA Document and I hereby sign the CLA

ChronosMasterOfAllTime commented 4 months ago

recheck

sfc-gh-dszmolka commented 4 months ago

hi and thanks for your contribution! can we maybe revisit this when Arrow v16 is actually released and use the official release ? It's not available yet.

sfc-gh-dszmolka commented 4 months ago

@ChronosMasterOfAllTime Arrow v16 looks to be released now, can you please modify your PR to have the official released v16.0.0 ?

ChronosMasterOfAllTime commented 4 months ago

@ChronosMasterOfAllTime Arrow v16 looks to be released now, can you please modify your PR to have the official released v16.0.0 ?

@sfc-gh-dszmolka done!

sfc-gh-pfus commented 4 months ago

Fantastic! Let's give it a few days to see if there is no critical update on Arrow side. Thanks a lot @ChronosMasterOfAllTime !

sfc-gh-pfus commented 4 months ago

It turns out, that Arrow 16 requires newer version of Go (1.21). We still support 1.19 so we have to put it on hold.

Yifeng-Sigma commented 4 months ago

It turns out, that Arrow 16 requires newer version of Go (1.21). We still support 1.19 so we have to put it on hold.

do we have a rough timeline of when we will do this upgrade?

sfc-gh-pfus commented 4 months ago

We (with @sfc-gh-sthyagaraj , our PM) are still working on all aspects (including legal) of deprecating old language versions. I can't make the promises, but I've been pushing this issue for a long time...