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

Add test CI: Docker based #5

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

Add test CI: Docker based #5

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

Comments

@kou
Copy link
Member

kou commented Aug 29, 2024

This part: https://github.com/apache/arrow/blob/58415d1fac50cb829b3dcf08526033d6db8c30db/.github/workflows/go.yml#L49-L163
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 9, 2024
@kou
apacheGH-5: Add test CI: Docker based
eb73e57 
Fix apacheGH-5

This part: https://github.com/apache/arrow/blob/58415d1fac50cb829b3dcf08526033d6db8c30db/.github/workflows/go.yml#L49-L135
kou added a commit to kou/arrow-go that referenced this issue Sep 9, 2024
@kou
apacheGH-5: Add test CI: Docker based
b694b95 
Fix apacheGH-5

This part: https://github.com/apache/arrow/blob/58415d1fac50cb829b3dcf08526033d6db8c30db/.github/workflows/go.yml#L49-L135
@kou kou mentioned this issue Sep 9, 2024
Merged
kou added a commit to kou/arrow-go that referenced this issue Sep 9, 2024
@kou
apacheGH-5: Add test CI: Docker based
1665004 
Fix apacheGH-5

This part: https://github.com/apache/arrow/blob/58415d1fac50cb829b3dcf08526033d6db8c30db/.github/workflows/go.yml#L49-L135
@raulcd raulcd closed this as completed in fbf0d2d Sep 9, 2024
@kou kou mentioned this issue Sep 9, 2024
Closed
@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-5: Add test CI: Docker based kou/arrow-go
2 participants
@kou @zeroshade