Skip to content

Conversation

@Akanksha-kedia
Copy link
Contributor

@Akanksha-kedia Akanksha-kedia commented Sep 30, 2025

Description

src/main/java/io/trino/gateway/ha/clustermonitor/ClusterStatsJdbcMonitor.java

  • Change: Fix silent failure on invalid backend URL configuration
  • Impact: Fail-fast behavior with descriptive error messages
  • Lines: Replace return clusterStats.build() with throw new IllegalArgumentException()

src/test/java/io/trino/gateway/ha/security/TestOIDC.java

  • Change: Replace System.out.println() with proper logging
  • Impact: Professional logging practices in test code
  • Lines: Add Logger import and replace print statement with log.info()

Additional context and related issues

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Sep 30, 2025
@ebyhr ebyhr merged commit 487137c into trinodb:main Oct 1, 2025
3 checks passed
@github-actions github-actions bot added this to the 17 milestone Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants