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

SNOW-679254: Properly support arrays as returned type #674

Open
Thomasdezeeuw opened this issue Oct 17, 2022 · 5 comments
Open

SNOW-679254: Properly support arrays as returned type #674

Thomasdezeeuw opened this issue Oct 17, 2022 · 5 comments
Assignees
Labels
enhancement The issue is a request for improvement or a new feature status-triage_done Initial triage done, will be further handled by the driver team

Comments

@Thomasdezeeuw
Copy link

Issue description

Currently arrays are returned a strings, which is not ideal because that means we have to parse those manually.

@github-actions github-actions bot changed the title Properly support arrays are returned type SNOW-679254: Properly support arrays are returned type Oct 17, 2022
@Thomasdezeeuw Thomasdezeeuw changed the title SNOW-679254: Properly support arrays are returned type SNOW-679254: Properly support arrays as returned type Nov 11, 2022
@yevgenypats
Copy link

Hi there! I also stumbled upon this and I would add that it's not a question of ideal or not but rather there is no way to parse as if there is something like "," inside a string saved in a string array you wont be able to distinguish between what is part of the array and what is part of the string in go.

@sfc-gh-dszmolka
Copy link
Contributor

thank you for raising this issue - even bigger thank you to yevgenypats for answering it. closing it for now, but if there's a PR which could address the requirement I'm happy to reopen to move forward

@Thomasdezeeuw
Copy link
Author

Thomasdezeeuw commented Mar 28, 2023

@sfc-gh-dszmolka did you read @yevgenypats comment? He agrees with me adding that's not possible to parse the array as text.

@Thomasdezeeuw
Copy link
Author

@sfc-gh-dszmolka can you reopen this as it's not solved.

@sfc-gh-dszmolka
Copy link
Contributor

reopening the issue and marking it as an enhancement for the product team to pick up in a later time

@sfc-gh-dszmolka sfc-gh-dszmolka added the enhancement The issue is a request for improvement or a new feature label Apr 3, 2023
@sfc-gh-dszmolka sfc-gh-dszmolka added the status-triage_done Initial triage done, will be further handled by the driver team label Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement The issue is a request for improvement or a new feature status-triage_done Initial triage done, will be further handled by the driver team
Projects
None yet
Development

No branches or pull requests

4 participants