Skip to content

Getting values for specific columns instead of trying to fetch all co…#586

Merged
sfc-gh-ogarciabarquero merged 1 commit intomainfrom
585-snow-2128660-getting-column-information-fails-describe-table-returning-less-columns
May 30, 2025
Merged

Getting values for specific columns instead of trying to fetch all co…#586
sfc-gh-ogarciabarquero merged 1 commit intomainfrom
585-snow-2128660-getting-column-information-fails-describe-table-returning-less-columns

Conversation

@sfc-gh-ogarciabarquero
Copy link
Collaborator

@sfc-gh-ogarciabarquero sfc-gh-ogarciabarquero commented May 29, 2025

…lumns.

Trying to get all columns generates errors when parameters enable extra columns for describe table command.

Please answer these questions before submitting your pull requests. Thanks!

  1. What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    Fixes SNOW-2128660

  2. Fill out the following pre-review checklist:

    • I am adding a new automated test(s) to verify correctness of my new code
    • I am adding new logging messages
    • I am adding new credentials
    • I am adding a new dependency
  3. Please describe how your code solves the related issue.

    Previously, the code was trying to fetch all columns for a table, which generated errors when parameters enabled extra columns for the describe table command. New code gets the values by index for needed columns only

…lumns.

Trying to get all columns generates errors when parameters enable extra columns for describe table command.
@sfc-gh-snowflakedb-snyk-sa
Copy link

sfc-gh-snowflakedb-snyk-sa commented May 29, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

Copy link
Collaborator

@sfc-gh-jvasquezrojas sfc-gh-jvasquezrojas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot locked and limited conversation to collaborators May 30, 2025
@sfc-gh-ogarciabarquero sfc-gh-ogarciabarquero merged commit 8c9784e into main May 30, 2025
67 of 90 checks passed
@sfc-gh-ogarciabarquero sfc-gh-ogarciabarquero deleted the 585-snow-2128660-getting-column-information-fails-describe-table-returning-less-columns branch May 30, 2025 19:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SNOW-2128680: [SNOW-2128660 ] Getting column information fails (Describe table returning less columns)

3 participants