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-853533: to_local_iterator doesn't support generating rows as case-insensitive #926

Closed
orrdermer1 opened this issue Jun 29, 2023 · 1 comment · Fixed by #1096
Closed
Labels
feature New feature or request triaged

Comments

@orrdermer1
Copy link

Please answer these questions before submitting your issue. Thanks!

  1. What version of Python are you using?

Python 3.10.8 (main, Oct 13 2022, 09:48:40) [Clang 14.0.0 (clang-1400.0.29.102)]

  1. What operating system and processor architecture are you using?

macOS-13.3.1-arm64-arm-64bit

  1. What are the component versions in the environment (pip freeze)?

... (Snowpark 1.5.1)

  1. What did you do?
df.collect(case_sensitive=False)
df.to_local_iterator()  # No such flag
  1. What did you expect to see?

Support for constructing case-insensitive rows (the underlying function, self._session._conn.execute supports the combination of to_iter=True and case_sensitive=False, looks like just an API addition)

  1. Can you set logging to DEBUG and collect the logs?

...

@orrdermer1 orrdermer1 added bug Something isn't working needs triage Initial RCA is required labels Jun 29, 2023
@github-actions github-actions bot changed the title to_local_iterator doesn't support generating rows as case-insensitive SNOW-853533: to_local_iterator doesn't support generating rows as case-insensitive Jun 29, 2023
@sfc-gh-stan sfc-gh-stan added feature New feature or request and removed bug Something isn't working needs triage Initial RCA is required labels Jun 30, 2023
@sfc-gh-aling
Copy link
Contributor

thanks for reaching out, I think this should be a straight forward change.
feel free to make a PR for it, we love community contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request triaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants