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-844431: Return false for unsupported locatorsUpdateCopy() function in SnowflakeDatabaseMetadata.java #1440

Merged
merged 1 commit into from
Jun 27, 2023

Conversation

sfc-gh-ext-simba-lb
Copy link
Contributor

@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb commented Jun 22, 2023

Overview

SNOW-844431

External contributors - please answer these questions before submitting a pull request. Thanks!

If an application makes a call to DatabaseMetadata describe function locatorsUpdateCopy(), a SnowflakeLoggedFeatureNotSupportedException is thrown and causes cascading failures.

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

    Fixes Should not throw exceptions from certain methods in net.snowflake.client.jdbc.SnowflakeDatabaseMetaData #1416
    https://github.com/snowflakedb/snowflake-sdks-drivers-issues-teamwork/issues/475

  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 modifying authorization mechanisms
    • I am adding new credentials
    • I am modifying OCSP code
    • I am adding a new dependency
  3. Please describe how your code solves the related issue.

Return false for this function instead of exception.

Pre-review checklist

  • This change has passed precommit
  • I have reviewed code coverage report for my PR in (Sonarqube)

@sfc-gh-igarish

This comment was marked as outdated.

@sfc-gh-ext-simba-lb

This comment was marked as outdated.

@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb changed the title SNOW-844431: Return reasonable value for unsupported describe functions in SnowflakeDatabaseMetadata.java SNOW-844431: Return false for unsupported locatorsUpdateCopy() function in SnowflakeDatabaseMetadata.java Jun 27, 2023
@sonarqubemergegate
Copy link

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Collaborator

@sfc-gh-igarish sfc-gh-igarish left a comment

Choose a reason for hiding this comment

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

LGTM

…ead of exception

Revert "return reasonable value for describe database metadata functions instead of exception"

This reverts commit e06df35.

return false for locatorsUpdateCopy()
@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb enabled auto-merge (squash) June 27, 2023 22:11
@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb merged commit ab78d97 into master Jun 27, 2023
12 checks passed
@sfc-gh-ext-simba-lb sfc-gh-ext-simba-lb deleted the SNOW-844431 branch June 27, 2023 23:05
@github-actions github-actions bot locked and limited conversation to collaborators Jun 27, 2023
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.

Should not throw exceptions from certain methods in net.snowflake.client.jdbc.SnowflakeDatabaseMetaData
2 participants