Skip to content

Conversation

@kentoyoshida
Copy link
Contributor

Summary

This PR adds a new jdbc_tracing configuration option that allows users to control the verbosity of Snowflake JDBC driver logs. This enhancement provides better debugging capabilities and allows users to reduce log noise in production environments.

Changes Made

  • Configuration Option: Added jdbc_tracing parameter with default value "INFO"
  • Valid Values: OFF, SEVERE, WARNING, INFO, CONFIG, FINE, FINER, FINEST, ALL
  • Implementation: The option is passed directly to the Snowflake JDBC driver via the "tracing" property
  • Documentation: Updated README.md with comprehensive description of the new parameter

Why This Is Useful

  • Debugging: Developers can increase logging verbosity (e.g., FINE, FINER, FINEST) to troubleshoot connection issues, query performance, or other JDBC-related problems
  • Production Optimization: Users can reduce log verbosity (e.g., WARNING, SEVERE) to minimize log output in production environments
  • Flexibility: Provides granular control over JDBC driver logging without requiring code changes

Technical Details

The configuration is implemented by setting the tracing property on the JDBC connection properties, which is a standard Snowflake JDBC driver feature. The default value "INFO" maintains backward compatibility with existing behavior.

Test Plan

  • Verify the configuration option is properly parsed and applied to JDBC connections
  • Test with different logging levels (OFF, INFO, FINE, ALL) to confirm they affect log output appropriately
  • Ensure backward compatibility by testing without the new parameter (should default to "INFO")
  • Validate that invalid values are handled gracefully by the JDBC driver
  • Test in both development and production-like environments to verify log behavior

🤖 Generated with Claude Code

…r logging

- Add jdbc_tracing parameter to SnowflakePluginTask interface with default value "INFO"
- Set tracing property in JDBC connection properties based on configuration
- Update README documentation with new configuration option
- Allows users to control verbosity of Snowflake JDBC driver logs (OFF, SEVERE, WARNING, INFO, CONFIG, FINE, FINER, FINEST, ALL)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copy link

@r-uehara0219 r-uehara0219 left a comment

Choose a reason for hiding this comment

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

LGTM

@kentoyoshida kentoyoshida merged commit 9a8fa04 into main Aug 26, 2025
1 check passed
@kentoyoshida kentoyoshida deleted the add-jdbc-tracing-config branch August 26, 2025 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants