-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Refactor to enable support for optional JDBC parameters for all JDBC destinations #10421
Conversation
/test connector=connectors/destination-mysql
|
Hey @girarda, How do I supply it via creating destination? There's no such node in To finalize tables after sync one needs to insert into destination tables from |
@@ -131,9 131,9 @@ public static Database createDatabase(final String username, | |||
final String jdbcConnectionString, | |||
final String driverClassName, | |||
final SQLDialect dialect, | |||
final String connectionProperties) { | |||
final Map<String, String> connectionProperties) { |
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.
Are there some needed updates on this code
Line 63 in c7b8db7
"oracle.net.encryption_client=REQUIRED;" |
…destinations (#10421) * refactoring to allow testing * MySQLDestination uses connection property map instead of url arguments * Update jdbc destinations * A little more generic * reset to master * reset to master * move to jdbcutils * Align when multiline * Align when multiline * Update postgres to use property map * Move tests to AbstractJdbcDestinationTest * clean * Align when multiline * return property map * Add postgres tests * update clickhouse * reformat * reset * reformat * fix test * reformat * fix bug * Add mssql tests * refactor test * fix oracle destination test * oracle tests * fix redshift acceptance test * Pass string * Revert "Pass string" This reverts commit 6978217. * Double deserialization * Revert "Double deserialization" This reverts commit ee8d752. * try updating json_operations * Revert "try updating json_operations" This reverts commit c8022c2. * json parse * Revert "json parse" This reverts commit 11a6725. * Revert "Revert "Double deserialization"" This reverts commit 213f47a. * Revert "Revert "Revert "Double deserialization""" This reverts commit 6682245. * move to constant * Add comment * map can be constant * Add comment * move map * hide in method * no need to create new map * no need to create new map * no need to create new map * enably mysql test * Update changelogs * Update changelog * update changelog * Bump versions * bump version * disable dbt support * update spec * update other oracle tests * update doc * bump seed * fix source test * update seed spec file * fix expected spec
…1093) * [10033] Destination-Snowflake: added basic part for support oauth login mode * added basic logic for token refresh * Fixed code to support pooled connections * Hide DBT transformations in cloud (#10583) * Bump Airbyte version from 0.35.35-alpha to 0.35.36-alpha (#10584) Co-authored-by: timroes <[email protected]> * 🐛 Source Shopify: fix wrong field type for tax_exemptions (#10419) * fix(shopify): wrong type for tax_exemptions abandoned_checkouts customer tax_exemptions had the wrong field type * fix(shopify): wrong type for tax_exemptions abandoned_checkouts customer tax_exemptions had the wrong field type * bump connector version Co-authored-by: marcosmarxm <[email protected]> * Remove storybook-addon-styled-component-theme (#10574) * Helm Chart: Secure chart for best practices (#10000) * 🐛 Source FB Marketing: fix `execute_in_batch` when batch is bigger than 50 (#10588) * fix execute_in_batch * add tests * fix pre-commit config Co-authored-by: Sherif A. Nada <[email protected]> Co-authored-by: Eugene Kulak <[email protected]> Co-authored-by: Sherif A. Nada <[email protected]> * Bmoric/move flag check to handler (#10469) Move the feature flag checks to the handler instead of the configuration API. This could have avoid some bug related to the missing flag check in the cloud project. * Documented product release stages (#10596) * Set resource limits for connector definitions: api layer (#10482) * Updated link to product release stages doc (#10599) * Change the block logic and block after the job creation (#10597) This is changing the check to see if a connection exist in order to make it more performant and more accurate. It makes sure that the workflow is reachable by trying to query it. * Add timeout to connector pod init container command (#10592) * add timeout to init container command * add disk usage check into init command * fix up disk usage checking and logs from init entrypoint * run format * fix orchestrator restart problem for cloud (#10565) * test time ranges for cancellations * try with wait * fix cancellation on worker restart * revert for CI testing that the test fails without the retry policy * revert testing change * matrix test the different possible cases * re-enable new retry policy * switch to no_retry * switch back to new retry * paramaterize correctly * revert to no-retry * re-enable new retry policy * speed up test fixees * significantly speed up test * fix ordering * use multiple task queues in connection manager test * use versioning for task queue change * remove sync workflow registration for the connection manager queue * use more specific example * respond to parker's comments * Fix the toggle design (#10612) * Source Hubspot: cast timestamp to date/datetime (#10576) * cast timestamp to date * change test name * fix corner cases * fix corner cases 2 * format code * changed method name * add return typing * bump version * updated spec and def yaml Co-authored-by: auganbay <[email protected]> * Update _helpers.tpl (#10617) as helm templates integers as float64, when using %d, it renders the value of external airbyte.minio.endpoint to "S3_MINIO_ENDPOINT: "http://minio-service:%!d(float64=9500)", therefore needed to be changed to %g * 🎉 Source Survey Monkey: add option to filter survey IDs (#8768) * Add custom survey_ids * bump version * Update survey_question schema * Add changelog * Allow null objects * merge master and format * Make all types safe with NULL and add survey_ids to all streams * Make additional types safe with NULL * Make additional types safe with NULL * One last safe NULL type * small fixes * solve conflic * small fixes * revert fb wrong commit * small fb correction * bump connector version Co-authored-by: marcosmarxm <[email protected]> * Fix doc links/loading (#10621) * Allow frontmatter in rendered markdown (#10624) * Adjust to new normalization name (#10626) * sweep pods from end time not start time (#10614) * Source Pinterest: fix typo in schema fields (#10223) * 🎉 add associations companies to deals, ticket and contacts stream (from PR 9027) (#10631) * Added associations to some CRM Object streams in Hubspot connector * Added associations in the relevant schemas * fix eof * bump connector version Co-authored-by: ksoenandar <[email protected]> * Source Chargebee: add transaction stream (#10312) * added transactions model * changes * fix * few changes * fix * added new stream in configured_catalog*.json * changes * removed new stream in configured_catalog*.json * solve small schema issues * add eof * bump connector version Co-authored-by: marcosmarxm <[email protected]> Co-authored-by: Marcos Marx <[email protected]> * Add missing continue as new (#10636) * Bump Airbyte version from 0.35.36-alpha to 0.35.37-alpha (#10640) Co-authored-by: benmoriceau <[email protected]> * exclude workers test from connectors builds on CI (#10615) * 🎉 Source Google Workspace Admin Reports: add support for Google Meet Audit Activity Events (#10244) * source(google-workspace-admin-reports): add support for Google Meet Audit activity events Signed-off-by: Michele Zuccala <[email protected]> * remove required fields * bump connector version * run format Co-authored-by: marcosmarxm <[email protected]> * stabilize connection manager tests (#10606) * stabilize connection manager tests * just call shutdown once * another run just so we can see if it's passing * another run just so we can see if it's passing * re-disable test * run another test * run another test * run another test * run another test * Log pod state if init pod wait condition times out (for debugging transient test issue) (#10639) * log pod state if init pod search times out * increase test timeout from 5 to 6 minutes to give kube pod process timeout time to trigger * format * upgrade gradle from 7.3.3 -> 7.4 (#10645) * upgrade temporal sdk to 1.8.1 (#10648) * upgrade temporal from mostly 1.6.0 to 1.8.1 * try bumping GSM to get newer grpc dep * Revert "try bumping GSM to get newer grpc dep" This reverts commit d837650. * upgrade temporal-testing as well * don't change version for temporal-testing-junit5 * 🎉 Source Google Ads: add network fields to click view stream * Google Ads #8331 - add network fields to click_view stream schema * Google Ads #8331 - add segments.ad_network_type to click_view pk according to PR review * Google Ads #8331 - bump version * Google Ads #8331 - update definition * Cloud Dashboard 1 (#10628) Publish metrics for: - created jobs tagged by release stage - failed jobs tagged by release stage - cancelled jobs tagged by release stage - succeed jobs tagged by release stage * Correct cancelled job metric name. (#10658) * Add attempt status by release stage metrics. (#10659) Add, - attempt_created_by_release_stage - attempt_failed_by_release_stage - attempt_succeeded_by_release_stage * 🐛 Source CockroachDB: fix connector replication failure due to multiple open portals error (#10235) * fix cockroachdb connector replication failure due to multiple open portals error * bump connector version Co-authored-by: marcosmarxm <[email protected]> * 🐙 octavia-cli: implement `generate` command (#10132) * Add try catch to make sure all handlers are closed (#10627) * Add try catch to make sure all handlers are closed * Handle exceptions while initializing writers * Bumpversion of connectors * bumpversion in seed * Fix bigquery denormalized tests * bumpversion seed of destination bigquery denormalized * Fix links in onboarding page (#10656) * Fix missing key inside map * Fix onboarding progress links * Add use-case links to onboarding (#10657) * Add use-case links to onboarding * Add new onboarding links * Set resource limits for connector definitions: expose in worker (#10483) * pipe through to worker * wip * pass source and dest def resource reqs to job client * fix test * use resource requirements utils to get resource reqs for legacy and new impls * undo changes to pass sync input to container launcher worker factory * remove import * fix hierarchy order of resource requirements * add nullable annotations * undo change to test * format * use destination resource reqs for normalization and make resource req utils more flexible * format * refactor resource requirements utils and add tests * switch to storing source/dest resource requirements directly on job sync config * fix tests and javadocs * use sync input resource requirements for container orchestrator pod * do not set connection resource reqs to worker reqs * add overrident requirement utils method test comment Co-authored-by: lmossman <[email protected]> * add mocks to tests * Bump Airbyte version from 0.35.37-alpha to 0.35.38-alpha (#10668) Co-authored-by: lmossman <[email protected]> * 🎉 Source Salesforce: speed up discovery >20x by leveraging parallel API calls (#10516) * 📖 improve salesforce docs & reorder properties in the spec (#10679) * Bump Airbyte version from 0.35.38-alpha to 0.35.39-alpha (#10680) Co-authored-by: sherifnada <[email protected]> * Improve note in salesforce docs about creating a RO user * Upgrade plop in connector generators (#10578) * Upgrade plop * Remove scaffolded code * Build fixes * Remove scaffolded code * Revert "Remove scaffolded code" This reverts commit 3911f52. * Revert "Remove scaffolded code" This reverts commit 549f790. * Remove .gitignore changes * Remove .gitignore changes * Update scaffold generated code * Replace titleCase with capitalCase (#10654) * Add capitalCase helper * Replace titleCase with capitalCase * Update generated scaffold files Co-authored-by: LiRen Tu <[email protected]> * 🐛 Fix toggle styling (#10684) * Fix error NPE in metrics emission. (#10675) * Fix missing type=button (#10683) * close ssh in case of exception during check in Postgres connector (#10620) * close ssh in case of exception * remove unwanted change * remove comment * format * do not close scanner * fix semi-colon * format * Refactor to enable support for optional JDBC parameters for all JDBC destinations (#10421) * refactoring to allow testing * MySQLDestination uses connection property map instead of url arguments * Update jdbc destinations * A little more generic * reset to master * reset to master * move to jdbcutils * Align when multiline * Align when multiline * Update postgres to use property map * Move tests to AbstractJdbcDestinationTest * clean * Align when multiline * return property map * Add postgres tests * update clickhouse * reformat * reset * reformat * fix test * reformat * fix bug * Add mssql tests * refactor test * fix oracle destination test * oracle tests * fix redshift acceptance test * Pass string * Revert "Pass string" This reverts commit 6978217. * Double deserialization * Revert "Double deserialization" This reverts commit ee8d752. * try updating json_operations * Revert "try updating json_operations" This reverts commit c8022c2. * json parse * Revert "json parse" This reverts commit 11a6725. * Revert "Revert "Double deserialization"" This reverts commit 213f47a. * Revert "Revert "Revert "Double deserialization""" This reverts commit 6682245. * move to constant * Add comment * map can be constant * Add comment * move map * hide in method * no need to create new map * no need to create new map * no need to create new map * enably mysql test * Update changelogs * Update changelog * update changelog * Bump versions * bump version * disable dbt support * update spec * update other oracle tests * update doc * bump seed * fix source test * update seed spec file * fix expected spec * Fix trial period time frame (#10714) * Bmoric/restore update with temporal (#10713) Restore the missing update call to temporal. It was making the update of a schedule to not be effective immediately. * Bump Airbyte version from 0.35.39-alpha to 0.35.40-alpha (#10716) Co-authored-by: benmoriceau <[email protected]> * Fix CockroachDbSource compilation error (#10731) * Fix CockroachDbSource compilation error * fix test too * 🎉 Source Zendesk: sync rate improvement (#9456) * Update Source Zendesk request execution with future requests. * Revert "Update Source Zendesk request execution with future requests." This reverts commit 2a3c1f8. * Add futures stream logics. * Fix stream * Fix full refresh streams. * Update streams.py. Fix all streams. Updated schema. * Add future request unit tests * Post review fixes. * Fix broken incremental streams. Fix SAT. Remove odd unit tests. * Comment few unit tests * Bump docker version * CDK: Ensure AirbyteLogger is thread-safe using Lock (#9943) * Ensure AirbyteLogger is thread-safe - Introduce a global lock to ensure `AirbyteLogger` is thread-safe. - The `logging` module is thread-safe, however `print` is not, and is currently used. This means that messages sent to stdout can clash if connectors use threading. This is obviously a huge problem when the IPC between the source/destination is stdout! - A `multiprocessing.Lock` could have been introduced however given that `logging` module is not multiprocess-safe I thought that thread-safety should be first goal. - IMO the `AirbyteLogger` should be a subclass of the `logging.Logger` so you have thread-safety automatically, however I didn't want to make a huge wholesale change here. * Revert lock and add deprecation warning instead * remove --cpu-shares flag (#10738) * Bump Airbyte version from 0.35.40-alpha to 0.35.41-alpha (#10740) Co-authored-by: jrhizor <[email protected]> * Add Scylla destination to index (#10741) * Add scylla to destination_definitions * Add woocommerce source * Update definition id * Add icon * update docker repository * reset to master * fix version * generate spec * Update builds.md * run gradle format (#10746) * Bump Airbyte version from 0.35.41-alpha to 0.35.42-alpha (#10747) Co-authored-by: girarda <[email protected]> * Change offer amount * Fix back link on signup page (#10732) * Fix back link on signup page * Add and correct uiConfig links * 🎉 Source redshift: implement privileges check (#9744) * update postgres source version (#10696) * update postgres source version * update spec * fix[api]: nullable connection schedule (#10107) * fix[api] inconsistent casing on OperationID for Operations API (#10464) * #10307 Fixes inconsistent casing on OperationID for Operations API * update generated doc Co-authored-by: alafanechere <[email protected]> * Display numbers in usage per connection table (#10757) * Add connector stage to dropdown value (#10677) * Add connector stage to dropdown value * Remove line break from i18n message * Update snowflake destination docs for correct host (#10673) * Update snowflake destination docs for correct host * Update snowflake.md * Update README.md * Update spec.json * Update README.md * Update spec.json * Update README.md * Update snowflake.md * Update spec.json * Update spec.json * 📕 source salesforce: fix broken page anchor in spec.json & add guide for adding read only user (#10751) * 🎉 Source Facebook Marketing: add activities stream (#10655) * add facebook marketing activities stream * update incremental test * add overrides for activities specific logic * formatting * update readme docs * remove test limitation * update dockerfile airbyte version * correct tests * bump connector version in config module Co-authored-by: marcosmarxm <[email protected]> * Add a note about running only in dev mode on M1 (#10772) Macs with M1 chip can run Airbyte only in dev mode right now, so to make it clear, I added a note about it and moved the hint about M1 chips to the top of the section. * push failures to segment (#10715) * test: new failures metadata for segment tracking * new failures metadata for segment tracking failure_reasons: array of all failures (as json objects) for a job - for general analytics on failures main_failure_reason: main failure reason (as json object) for this job - for operational usage (for Intercom) - currently this is just the first failure reason chronologically - we'll probably to change this when we have more data on how to determine failure reasons more intelligently - added an attempt_id to failures so we can group failures by attempt - removed stacktrace from failures since it's not clear how we'd use these in an analytics use case (and because segment has a 32kb size limit for events) * remove attempt_id attempt info is already in failure metadata * explicitly sort failures array chronologically * replace "unknown" enums with null note: ImmutableMaps don't allow nulls * move sorting to the correct place * Update temporal retention TTL from 7 to 30 days (#10635) Increase the temporal retention to 30 days instead of 7. It will help with on call investigation. * Add count connection functions (#10568) * Add count connection functions * Fix new configRepository queries - Remove unnecessary joins - Fix countConnection * Use existing mock data for tests * Adds default sidecar cpu request and limit and add resources to the init container (#10759) * close ssh tunnel in case of exception in destination consumer (#10686) * close ssh tunnel in case of exception * format * fix salesforce docs markdown formatting * Fix typo in salesforce docs * Extract event from the temporal worker run factory (#10739) Extract of different events that can happen to a sync into a non temporal related interface. * Bump Airbyte version from 0.35.42-alpha to 0.35.43-alpha (#10778) Co-authored-by: sherifnada <[email protected]> * Added a note about running in dev mode on M1 macs (#10776) Currently, Macs with M1 chips can run Airbyte only in dev mode. I added a note about that. * Destination Snowflake: add missing version in changelog (#10779) * Hide shopify in Cloud (#10783) * Metrics Reporter Queries Part 1 (#10663) Add all the simpler queries from https://docs.google.com/document/d/11pEUsHyKUhh4CtV3aReau3SUG-ncEvy6ROJRVln6YB4/edit?usp=sharing. - Num Pending Jobs - Num Concurrent Jobs - Oldest Pending Job - Oldest Running Job * Bump Airbyte version from 0.35.43-alpha to 0.35.44-alpha (#10789) * Bump Airbyte version from 0.35.43-alpha to 0.35.44-alpha * Commit. * Add exception block. * Why would having try catch work? * Add logging to figure out. * Undo all debugging changes. * Better comments. Co-authored-by: davinchia <[email protected]> Co-authored-by: Davin Chia <[email protected]> * Update api-documentation.md * jdbc build fixes (#10799) * Update api-documentation.md * Exclude package.json from codeowners (#10805) * 🎉 Source Chargebee: add credit note model (#10795) * feat(chargebee) add credit note model * fix(airbyte): update version Dockerfile * fix(airbyte): update version Dockerfile v2 * Source Chargebee: run format and correct unit test (#10811) * feat(chargebee) add credit note model * fix(airbyte): update version Dockerfile * fix(airbyte): update version Dockerfile v2 * correct unit test Co-authored-by: Koen Sengers <[email protected]> * 🎉 Source Chartmogul: Add CustomerCount stream (#10756) * 🎉 Source Chartmogul: Add CustomerCount stream * Update description * address comments * update changelog * format source file * run seed file Co-authored-by: marcosmarxm <[email protected]> * default to no resource limits for OSS (#10800) * Add autoformat (#10808) * Bump Airbyte version from 0.35.44-alpha to 0.35.45-alpha (#10818) Co-authored-by: lmossman <[email protected]> * Set default values as current values in editMode (#10486) * Set default values as current values in editMode * Fix unit tests * Save signup fields (#10768) * Temporary save signup fields into firebase_user.displayName * Use default values if no displayName was stored before * Move regsiter to localStorage * Address PR comments * Source Woocommerce: fixes (#10529) * fixed issues * Fix: multiple issues * modify configured catalog * Fix: remove unused variables * Fix: orders request with parameters * Fix: add new line in configured catalogs * Fix: remove unused imports * Fix: catalog changes * Source woocommerce: publishing connector (#10791) * fixed issues * Fix: multiple issues * modify configured catalog * Fix: remove unused variables * Fix: orders request with parameters * Fix: add new line in configured catalogs * Fix: remove unused imports * Fix: catalog changes * fix: change schema for meta_data Co-authored-by: Manoj <[email protected]> * Surface any active child thread of dying connectors (#10660) * Interrupt child thread of dying connectors to avoid getting stuck * Catch and print stacktrace * Add test on interrupt/kill time outs * Send message to sentry too * Add another token to alleviate API limit pressure. (#10826) We are running into Github API rate limits. This PR: - introduces another token as a temp solution. - reorganises the workflow file. * Add caching to all jobs in the main build. (#10801) Add build dependency caching to all jobs in the main build. This speeds things up by 5 mins over the previously uncached time. * 🐛 Handle try/catch in BigQuery destination consumers (#10755) * Handle try/catch in BigQuery destination consumers * Remove parallelStream * Bumpversion of connector * update changelogs * update seeds * Format code (#10837) * Regenerate MySQL outputs from normalization tests * format * Use cypress dashboard and stabilize e2e tests (#10807) * Record e2e tests to cypress dashboard * Make env variable accessible in script * Improve e2e_test script * Properly wait for server to be ready * Isolate test suites better * More test isolation * Revert baseUrl for development * 🐛 Source Github: add new streams `Deployments`, `ProjectColumns`, `PullRequestCommits` (#10385) Signed-off-by: Sergey Chvalyuk <[email protected]> * Remove the use of ConfigPersistence for ActorCatalog operation (#10387) * Skip ConfigPersistence for ActorCatalog operations * Fix catalog insertion logic - ActorCatalog and ActorCatalogFetchEvent are stored within the same transation. - The function writing catalog now automatically handles deduplication. - Fixed function visibility: helper function to handle ActorCatalog insertion are now private. * Fix fetch catalog query take the catalog associated with the latest fetch event in case where multiple event are present for the same config, actorId, actor version. * Fix name of columns used for insert * Add testing on deduplication of catalogs * Add javadoc for actor catalog functions * Rename sourceId to actorId * Fix formatting * Update integrations README.md (#10851) Updated verbiage from grades to stages Updated connector stages to match cloud stage tags Added connectors missing on README.md that appear on cloud drop down * [10033] Destination-Snowflake: added basic part for support oauth login mode * added basic logic for token refresh * Updated spec to support DBT normalization and OAuth * snowflake oauth Signed-off-by: Sergey Chvalyuk <[email protected]> * test_transform_snowflake_oauth added Signed-off-by: Sergey Chvalyuk <[email protected]> * [4654] Added backward compatibility * Added test to check a backward compatibility * fixed oauth connection * Updated doc, fixed code as per comments in PR * to be more explicit Signed-off-by: Sergey Chvalyuk <[email protected]> * Added executor service * Fixed merge conflict * Updated doc and bumped version * Bumped version * bump 0.1.71 -> 0.1.72 Signed-off-by: Sergey Chvalyuk <[email protected]> * Updated doc * fix version in basic-normalization.md Signed-off-by: Sergey Chvalyuk <[email protected]> * Added explicit re-set property, but even now it already works * dummy bumping version * updated spec Co-authored-by: ievgeniit <[email protected]> Co-authored-by: Tim Roes <[email protected]> Co-authored-by: Octavia Squidington III <90398440 [email protected]> Co-authored-by: timroes <[email protected]> Co-authored-by: Philippe Boyd <[email protected]> Co-authored-by: marcosmarxm <[email protected]> Co-authored-by: Álvaro Torres Cogollo <[email protected]> Co-authored-by: Eugene Kulak <[email protected]> Co-authored-by: Sherif A. Nada <[email protected]> Co-authored-by: Eugene Kulak <[email protected]> Co-authored-by: Benoit Moriceau <[email protected]> Co-authored-by: Amruta Ranade <11484018 [email protected]> Co-authored-by: Charles <[email protected]> Co-authored-by: Parker Mossman <[email protected]> Co-authored-by: Jared Rhizor <[email protected]> Co-authored-by: augan-rymkhan <93112548 [email protected]> Co-authored-by: auganbay <[email protected]> Co-authored-by: keterslayter <32784192 [email protected]> Co-authored-by: Daniel Diamond <33811744 [email protected]> Co-authored-by: Ronald Fortmann <72810611 [email protected]> Co-authored-by: Marcos Marx <[email protected]> Co-authored-by: ksoenandar <[email protected]> Co-authored-by: Aaditya Sinha <75474786 [email protected]> Co-authored-by: benmoriceau <[email protected]> Co-authored-by: Michele Zuccala <[email protected]> Co-authored-by: vitaliizazmic <75620293 [email protected]> Co-authored-by: Davin Chia <[email protected]> Co-authored-by: Lakshmikant Shrinivas <[email protected]> Co-authored-by: Augustin <[email protected]> Co-authored-by: Christophe Duong <[email protected]> Co-authored-by: lmossman <[email protected]> Co-authored-by: lmossman <[email protected]> Co-authored-by: Maksym Pavlenok <[email protected]> Co-authored-by: sherifnada <[email protected]> Co-authored-by: LiRen Tu <[email protected]> Co-authored-by: Subodh Kant Chaturvedi <[email protected]> Co-authored-by: girarda <[email protected]> Co-authored-by: Vadym Hevlich <[email protected]> Co-authored-by: jdclarke5 <[email protected]> Co-authored-by: jrhizor <[email protected]> Co-authored-by: girarda <[email protected]> Co-authored-by: Azhar Dewji <[email protected]> Co-authored-by: Alasdair Brown <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Lucas Wiley <[email protected]> Co-authored-by: Philip Corr <[email protected]> Co-authored-by: Greg Solovyev <[email protected]> Co-authored-by: Peter Hu <[email protected]> Co-authored-by: Malik Diarra <[email protected]> Co-authored-by: Thibaud Chardonnens <[email protected]> Co-authored-by: davinchia <[email protected]> Co-authored-by: Erica Struthers <93952107 [email protected]> Co-authored-by: Edward Gao <[email protected]> Co-authored-by: Tim Roes <[email protected]> Co-authored-by: ksengers <30521298 [email protected]> Co-authored-by: Koen Sengers <[email protected]> Co-authored-by: Titas Skrebe <[email protected]> Co-authored-by: Artem Astapenko <3767150 [email protected]> Co-authored-by: Manoj Reddy KS <[email protected]> Co-authored-by: Harshith Mullapudi <[email protected]> Co-authored-by: Juan <80164312 [email protected]>
What
AbstractJdbcDestination
to streamline support for optional JDBC parametersHow
Add an abstract method
protected abstract Map<String, String> getDefaultConnectionProperties(final JsonNode config);
which concrete destination can implement to define default parameters that need to be set.
When building a connection, parse the jdbc url parameters and add the parameters to the default ones, making sure the default parameters are not being overridden.
Create the Database using a property map instead of a formatting string. This step is not strictly required, but makes the flow easier to follow.
Recommended reading order
airbyte-integrations/connectors/destination-jdbc/src/main/java/io/airbyte/integrations/destination/jdbc/AbstractJdbcDestination.java
: This is where the core of the logic is. The new code is pretty much just moved fromMySQLDestination
as a way to generalize the solutionairbyte-integrations/connectors/destination-mysql/src/main/java/io/airbyte/integrations/destination/mysql/MySQLDestination.java
: A lot of code was moved toAbstractJdbcDestination
andallowLoadLocalInfile
.airbyte-db/lib/src/main/java/io/airbyte/db/jdbc/JdbcUtils.java
implements the jdbc parameters parsing to a Map<String, String>airbyte-db/lib/src/main/java/io/airbyte/db/Databases.java
: Removed the option to pass in connectionProperties as a String in favor of always passing a Map<String, String>airbyte-integrations/connectors/destination-clickhouse/src/main/java/io/airbyte/integrations/destination/clickhouse/ClickhouseDestination.java
: Pass default parameters as connection properties instead of embedding them in the URLairbyte-integrations/connectors/destination-mariadb-columnstore/src/main/java/io/airbyte/integrations/destination/mariadb_columnstore/MariadbColumnstoreDestination.java
: Pass default parameters as connection properties instead of embedding them in the URLairbyte-integrations/connectors/destination-postgres/src/main/java/io/airbyte/integrations/destination/postgres/PostgresDestination.java
: Pass default parameters as connection properties instead of embedding them in the URLairbyte-integrations/connectors/destination-redshift/src/main/java/io/airbyte/integrations/destination/redshift/RedshiftInsertDestination.java
: Pass default parameters as connection properties instead of embedding them in the URLairbyte-integrations/connectors/destination-mssql/src/main/java/io/airbyte/integrations/destination/mssql/MSSQLDestination.java
: Pass default parameters as connection properties instead of embedding them in the URLairbyte-integrations/connectors/destination-oracle/src/main/java/io/airbyte/integrations/destination/oracle/OracleDestination.java
: Pass default parameters as connection properties instead of embedding them in the URLairbyte-integrations/connectors/destination-snowflake/src/main/java/io/airbyte/integrations/destination/snowflake/SnowflakeInsertDestination.java
: ImplementgetDefaultConnectionProperties
airbyte-integrations/connectors/destination-snowflake/src/main/java/io/airbyte/integrations/destination/snowflake/SnowflakeInternalStagingDestination.java
: ImplementgetDefaultConnectionProperties
airbyte-integrations/connectors/source-jdbc/src/main/java/io/airbyte/integrations/source/jdbc/AbstractJdbcSource.java
: parse connection_properties string to a map of properties before callingcreateStreamingJdbcDatabase
In particular, I want to point out of the following:
SnowflakeInsertDestination
andSnowflakeInternalStagingDestination
don't follow the same pattern as the other connectors🚨 User Impact 🚨
Are there any breaking changes? What is the end result perceived by the user? If yes, please merge this PR with the 🚨🚨 emoji so changelog authors can further highlight this if needed.
Pre-merge Checklist
Expand the relevant checklist and delete the others.
Updating a connector
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampleAirbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing/publish
command described here