Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update license information #3

Closed
kou opened this issue Aug 29, 2024 · 0 comments · Fixed by #130
Closed

Update license information #3

kou opened this issue Aug 29, 2024 · 0 comments · Fixed by #130
Milestone
18.0.0

Comments

@kou
Copy link
Member

kou commented Aug 29, 2024

It seems that we can simplify https://github.com/apache/arrow/blob/main/go/LICENSE.txt because it has unrelated notes like: https://github.com/apache/arrow/blob/58415d1fac50cb829b3dcf08526033d6db8c30db/go/LICENSE.txt#L204-L240
kou pushed a commit that referenced this issue Aug 30, 2024
@richardartoul @kou
ARROW-7921: [Go] Add Reset method to various components and clean up …
e324c33 
…comments.

The reset method allow the data structures to be re-used so they don't have to be allocated over and over again.

Closes #6430 from richardartoul/ra/merge-upstream and squashes the following commits:

5a0828187 <Richard Artoul> Add license to test file
d76be05ca <Richard Artoul> Add test for data reset
d102b1fff <Richard Artoul> Add tests
d3e6e6785 <Richard Artoul> cleanup comments
c8525aece <Richard Artoul> Add Reset method to int array (#5)
489ca2593 <Richard Artoul> Fix array.setData() to retain before release (#4)
88cd05ff8 <Richard Artoul> Add reset method to Data (#3)
6d1b2775c <Richard Artoul> Add Reset() method to String array (#2)
dca230383 <Richard Artoul> Add Reset method to buffer and cleanup comments (#1)

Lead-authored-by: Richard Artoul <richard.artoul@datadoghq.com>
Co-authored-by: Richard Artoul <richardartoul@gmail.com>
Signed-off-by: Sebastien Binet <binet@cern.ch>
kou added a commit to kou/arrow-go that referenced this issue Sep 17, 2024
@kou
apacheGH-3: Update license information
ca37352 
The current LICENSE.txt is based on apache/arrow's one. It includes
unrelated notes for go/.
kou added a commit to kou/arrow-go that referenced this issue Sep 17, 2024
@kou
apacheGH-3: Update license information
6ba61dc 
Fix apacheGH-3

The current LICENSE.txt is based on apache/arrow's one. It includes
unrelated notes for go/.
@kou kou mentioned this issue Sep 17, 2024
Merged
@kou kou closed this as completed in #130 Sep 17, 2024
kou added a commit that referenced this issue Sep 17, 2024
@kou
GH-3: Update license information (#130)
3e01c55 
Fix GH-3

The current LICENSE.txt is based on apache/arrow's one. It includes
unrelated notes for go/.
@zeroshade zeroshade added this to the 18.0.0 milestone Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
18.0.0
Development

Successfully merging a pull request may close this issue.

GH-3: Update license information kou/arrow-go
2 participants
@kou @zeroshade