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.

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

[Go][Builder] Add AppendFromText method #65

Closed
yevgenypats opened this issue Mar 22, 2023 · 4 comments
Closed

[Go][Builder] Add AppendFromText method #65

yevgenypats opened this issue Mar 22, 2023 · 4 comments
Labels
good-first-issue Good for newcomers Type: enhancement New feature or request

Comments

@yevgenypats
Copy link
Contributor

Describe the enhancement requested

AppendFromText can be useful in CSV parsing and various other scenarios. Full discussion in this PR.

Component(s)

Go

@yevgenypats yevgenypats added the Type: enhancement New feature or request label Mar 22, 2023
@zeroshade zeroshade added the good-first-issue Good for newcomers label Apr 25, 2023
@assignUser assignUser transferred this issue from apache/arrow Aug 30, 2024
@singh1203
Copy link
Contributor

Hello @yevgenypats, I would love to take on this work if you're not already handling it.
cc: @zeroshade

@zeroshade
Copy link
Member

I think this work has already been done with the AppendValueFromString method on the builder interface. Can you confirm? If so, I'll close this issue

@singh1203
Copy link
Contributor

I think this work has already been done with the AppendValueFromSt ring method on the builder interface. Can you confirm? If so, I'll close this issue

Sure! I'll confirm and report here. 👍

@singh1203
Copy link
Contributor

@zeroshade You were right. It's already been completed in this PR - 34986.
Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good-first-issue Good for newcomers Type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants