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
CDK Destinations: DB Metadata calls cleanup #34533
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️ the comment, super informative
55f3d50
to
ed4693e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpicky comments about comments, otherwise lgtm
airbyte-cdk/java/airbyte-cdk/core/src/main/java/io/airbyte/cdk/db/jdbc/DefaultJdbcDatabase.java
Outdated
Show resolved
Hide resolved
airbyte-cdk/java/airbyte-cdk/core/src/main/java/io/airbyte/cdk/db/jdbc/DefaultJdbcDatabase.java
Outdated
Show resolved
Hide resolved
9a96eb4
to
074d98e
Compare
Co-authored-by: Edward Gao <edward.gao@airbyte.io>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
c857478
to
5f1c097
Compare
* @return | ||
* @param <T> | ||
*/ | ||
public static <T> List<T> allOf(final List<CompletableFuture<T>> futures) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this file needed? not in use afaict, and the implementation looks broken
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created for something else but removed the impl for now on this branch. I'll remove it later if not needed.
airbyte-cdk/java/airbyte-cdk/airbyte-commons/src/main/java/io/airbyte/commons/json/Jsons.java
Outdated
Show resolved
Hide resolved
/publish-java-cdk
|
…34634) ## What * See #34533 * Resolves airbytehq/alpha-beta-issues#2811 * Resolves airbytehq/alpha-beta-issues#2808 * Resolves airbytehq/alpha-beta-issues#2804 * Resolves airbytehq/alpha-beta-issues#2800 * Resolves airbytehq/alpha-beta-issues#2796 ## How * Execute dbMetadata ResultSet calls within a connection's lifecycle try-resource statement.
…34634) ## What * See #34533 * Resolves airbytehq/alpha-beta-issues#2811 * Resolves airbytehq/alpha-beta-issues#2808 * Resolves airbytehq/alpha-beta-issues#2804 * Resolves airbytehq/alpha-beta-issues#2800 * Resolves airbytehq/alpha-beta-issues#2796 ## How * Execute dbMetadata ResultSet calls within a connection's lifecycle try-resource statement.
## What * Cleaning up unsafe metadata calls
…irbytehq#34634) ## What * See airbytehq#34533 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2811 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2808 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2804 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2800 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2796 ## How * Execute dbMetadata ResultSet calls within a connection's lifecycle try-resource statement.
## What * Cleaning up unsafe metadata calls
…irbytehq#34634) ## What * See airbytehq#34533 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2811 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2808 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2804 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2800 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2796 ## How * Execute dbMetadata ResultSet calls within a connection's lifecycle try-resource statement.
## What * Cleaning up unsafe metadata calls
…irbytehq#34634) ## What * See airbytehq#34533 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2811 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2808 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2804 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2800 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2796 ## How * Execute dbMetadata ResultSet calls within a connection's lifecycle try-resource statement.
## What * Cleaning up unsafe metadata calls
…irbytehq#34634) ## What * See airbytehq#34533 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2811 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2808 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2804 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2800 * Resolves https://github.com/airbytehq/alpha-beta-issues/issues/2796 ## How * Execute dbMetadata ResultSet calls within a connection's lifecycle try-resource statement.
What