Skip to content

Conversation

sergiyvamz
Copy link
Contributor

@sergiyvamz sergiyvamz commented Mar 28, 2024

Summary

fix: connection identification and tracking

Description

Additional Reviewers

@karenc-bq
@crystall-bitquill
@aaronchung-bitquill

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sergiyvamz sergiyvamz added the wip Pull request that is a work in progress label Mar 28, 2024
Copy link

github-actions bot commented Mar 28, 2024

Qodana Community for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked

View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

  1. Register at Qodana Cloud and configure the action
  2. Use GitHub Code Scanning with Qodana
  3. Host Qodana report at GitHub Pages
  4. Inspect and use qodana.sarif.json (see the Qodana SARIF format for details)

To get *.log files or any other Qodana artifacts, run the action with upload-result option set to true,
so that the action will upload the files as the job artifacts:

      - name: 'Qodana Scan'
        uses: JetBrains/[email protected]
        with:
          upload-result: true
Contact Qodana team

Contact us at [email protected]

@sergiyvamz sergiyvamz force-pushed the fix-connection-tracker branch from f29f401 to b5b75d2 Compare April 1, 2024 16:36
@sergiyvamz sergiyvamz force-pushed the fix-connection-tracker branch from 6053ee4 to bf09978 Compare April 1, 2024 16:46
@sergiyvamz sergiyvamz added ready for review Pull requests that are ready to be reviewed and removed wip Pull request that is a work in progress labels Apr 1, 2024
@sergiyvamz sergiyvamz changed the title [WIP] fix: connection identification and tracking fix: connection identification and tracking Apr 1, 2024
@sergiyvamz sergiyvamz merged commit f57a92e into main Apr 1, 2024
@sergiyvamz sergiyvamz deleted the fix-connection-tracker branch April 1, 2024 22:28
sergiyvamz added a commit that referenced this pull request Apr 10, 2024
…nProvider.java

Co-authored-by: Karen <[email protected]>

Update wrapper/src/main/java/software/amazon/jdbc/DataSourceConnectionProvider.java

Co-authored-by: Karen <[email protected]>

Update wrapper/src/main/java/software/amazon/jdbc/DataSourceConnectionProvider.java

Co-authored-by: Karen <[email protected]>

Update docs/using-the-jdbc-driver/using-plugins/UsingTheFailoverPlugin.md

Co-authored-by: crystall-bitquill <[email protected]>

Update docs/using-the-jdbc-driver/UsingTheJdbcDriver.md

Co-authored-by: crystall-bitquill <[email protected]>

code review

green node endpoint replacement

fix: connection identification and tracking (#943)
sergiyvamz added a commit that referenced this pull request Apr 10, 2024
…nProvider.java

Co-authored-by: Karen <[email protected]>

Update wrapper/src/main/java/software/amazon/jdbc/DataSourceConnectionProvider.java

Co-authored-by: Karen <[email protected]>

Update wrapper/src/main/java/software/amazon/jdbc/DataSourceConnectionProvider.java

Co-authored-by: Karen <[email protected]>

Update docs/using-the-jdbc-driver/using-plugins/UsingTheFailoverPlugin.md

Co-authored-by: crystall-bitquill <[email protected]>

Update docs/using-the-jdbc-driver/UsingTheJdbcDriver.md

Co-authored-by: crystall-bitquill <[email protected]>

code review

green node endpoint replacement

fix: connection identification and tracking (#943)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review Pull requests that are ready to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants