-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Arrow Go 18.0.0 release post #550
Add Arrow Go 18.0.0 release post #550
Conversation
#### Features | ||
|
||
* Added the initial implementation for the Decimal32/Decimal64 data types [Go-120](https://github.com/apache/arrow-go/issues/120) | ||
* Added `is_null`, `is_not_null` and `is_nan` function kernels to the `compute` package [GH-134](https://github.com/apache/arrow-go/issues/134) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: inconsistency over using Go-NNN
vs GH-NNN
vs GO-NNN
for arrow-go
issues
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The inconsistency is for whether the issue is available on arrow-go or were resolved in the monorepo before we did the move. Issues that were resolved before we moved the repo and don't exist in the arrow-go
repo are using GH while the issues which exist in the arrow-go
repo are using GO
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both issues here are on arrow-go, though, is what I'm saying. (Go-120 vs GH-134)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
accepted Kou's sugestions which alleviated the discrepency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exciting updates!
|
||
With this release, the minimum Go version is now 1.22 | ||
|
||
### Arrow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need the Arrow
header/would this look cleaner dropping one level of header?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've consistently used the separation of updates between the Arrow lib and the Parquet lib, so i was maintaining those headings as with previous posts.
Co-authored-by: David Li <[email protected]>
Co-authored-by: Bryce Mecum <[email protected]>
Co-authored-by: Bryce Mecum <[email protected]>
Co-authored-by: Sutou Kouhei <[email protected]>
Co-authored-by: Sutou Kouhei <[email protected]>
Co-authored-by: Sutou Kouhei <[email protected]>
Thanks everyone for the reviews! |
Adding a release post for the 18.0.0 release of Arrow Go, feel free to tag any additional reviewers I missed. Thanks!