🎉 Python CDK: Allow setting network adapter args on outgoing HTTP requests #4493

sherifnada · 2021-07-02T07:28:01Z

What

How

Adds a method request_kwargs which a developer can use to specify kwargs on the outgoing request.

sherifnada · 2021-07-02T20:32:26Z

/publish-cdk

❌ https://github.com/airbytehq/airbyte/actions/runs/994795348
❌ https://github.com/airbytehq/airbyte/actions/runs/994795348
❌ https://github.com/airbytehq/airbyte/actions/runs/994795348

…ose-kwargs-in-http-stream

…ytehq/airbyte into sherif/expose-kwargs-in-http-stream

sherifnada · 2021-07-02T20:53:02Z

/publish-cdk

❌ https://github.com/airbytehq/airbyte/actions/runs/994834110
❌ https://github.com/airbytehq/airbyte/actions/runs/994834110

sherifnada · 2021-07-02T21:34:53Z

/publish-cdk

❌ https://github.com/airbytehq/airbyte/actions/runs/994927908
❌ https://github.com/airbytehq/airbyte/actions/runs/994927908

sherifnada · 2021-07-02T21:35:08Z

.github/workflows/publish-cdk-command.yml

@@ -27,7 +27,7 @@ jobs:
      - name: Checkout Airbyte
        uses: actions/checkout@v2
      - name: Build CDK Package
-        run: ./gradlew --no-daemon :airbyte-cdk:python:build
+        run: ./gradlew --no-daemon --no-build-cache :airbyte-cdk:python:build


…ose-kwargs-in-http-stream

davinchia

Oh this is a great idea!

…ose-kwargs-in-http-stream

sherifnada · 2021-07-04T17:52:26Z

/publish-cdk

❌ https://github.com/airbytehq/airbyte/actions/runs/998850336
❌ https://github.com/airbytehq/airbyte/actions/runs/998850336

keu · 2021-07-04T21:15:35Z

airbyte-cdk/python/airbyte_cdk/sources/streams/http/http.py

@@ -190,9 +203,8 @@ def _send_request(self, request: requests.PreparedRequest) -> requests.Response:
        Unexpected transient exceptions use the default backoff parameters.
        Unexpected persistent exceptions are not handled and will cause the sync to fail.
        """
-        response: requests.Response = self._session.send(request)
+        response: requests.Response = self._session.send(request, **request_kwargs)


I see these changes as potentially destructive.
The problem is that we now provide multiple ways for users to define request parameters (headers, path, params, auth). If user will do this then he needs to reimplement the logic of request_params, request_headers, etc, as there is no way to know default values in request_kwargs call.

ok, I checked the source code and it seems that the only parameters that we can use here are:

allow_redirects

stream

timeout

verify

cert

proxies
So there is no conflict with request args, but it still not possible to customize auth (FYI)

keu · 2021-07-04T21:21:42Z

airbyte-cdk/python/airbyte_cdk/sources/streams/http/http.py

@@ -224,8 +236,8 @@ def read_records(
                params=self.request_params(stream_state=stream_state, stream_slice=stream_slice, next_page_token=next_page_token),
                json=self.request_body_json(stream_state=stream_state, stream_slice=stream_slice, next_page_token=next_page_token),
            )
-
-            response = self._send_request(request)
+            request_kwargs = self.request_kwargs(stream_state=stream_state, stream_slice=stream_slice, next_page_token=next_page_token)


not sure I understand why you split kwargs here? can't you pass it directly to PreparedRequest?

ok, I see you tried to mimic Session.request() logic, what is the reason to use PreparedRequest?
is it just to not call request_body_json and other functions multiple times in case of retry? what if this is what user wants? i.e. what if the request fails because of one of its parameters that need to be changed?

using Session.prepare_request instead of request.prepare() or just Session.request(), has benefit of sharing session states like cookies across requests.

we need to use prepared request for Session.send

good idea, changed to use Session.prepare_request

keu

some question raised

…ose-kwargs-in-http-stream

sherifnada · 2021-07-07T17:50:26Z

/publish-cdk

🕑 https://github.com/airbytehq/airbyte/actions/runs/1008946561
✅ https://github.com/airbytehq/airbyte/actions/runs/1008946561

sherifnada · 2021-07-07T17:58:21Z

/publish-cdk dry-run=false

🕑 https://github.com/airbytehq/airbyte/actions/runs/1008967123
✅ https://github.com/airbytehq/airbyte/actions/runs/1008967123

…uests (#4493)

* init the new connector source-zendesk-support * Finished a development of ZenDesk streams * Source ZenDesk: finished * Source ZenDesk: remove unused test files * Source ZenDesk: format and validate code * Source Zendesk: update docs * Remove unused files * add a stream_slices logic for ticket_comments stream * 🎉 Python CDK: Allow setting network adapter args on outgoing HTTP requests (#4493) * 🎉 Destination S3: support `anyOf` `allOf` and `oneOf` (#4613) * Support combined restrictions in json schema * Bump s3 version * Add more test cases * Update changelog * Add more test cases * Update documentation * Format code * SAT: verify `AIRBYTE_ENTRYPOINT` is defined (#4478) * save changes required for work; TODO locate all places that need to be updated to make test working * move new test inside test_spec * apply suggestions * change return type + add check env = space_joined_entrypoint * requested * add check entrypoint with env * bump SAT --version && changelog update * merge && fix changelog * changes * add dynamic docker runner creator + test having properties * update the names * change names * make fixtures * upd text * Update airbyte-integrations/bases/source-acceptance-test/unit_tests/test_spec_unit.py Co-authored-by: Eugene Kulak <[email protected]> * requested changes * Update airbyte-integrations/bases/source-acceptance-test/unit_tests/test_spec_unit.py Co-authored-by: Eugene Kulak <[email protected]> * Update airbyte-integrations/bases/source-acceptance-test/unit_tests/test_spec_unit.py Co-authored-by: Eugene Kulak <[email protected]> * apply requested changes * change names (requested) * move binary strings to standard with convertation in builder * fixing merge-conflict side effect Co-authored-by: Eugene Kulak <[email protected]> * Migrate Quickstart to use PokeAPI (#4615) * Migrate Quickstart to use PokeAPI * Words words words Co-authored-by: Abhi Vaidyanatha <[email protected]> * Left isn't right (#4616) Co-authored-by: Abhi Vaidyanatha <[email protected]> * Create on on-oci-vm.md (#4468) * Create on on-oci-vm.md Deployment guide for Airbyte on Oracle Cloud Infrastructure (OCI) VM * Update on-oci-vm.md Adding the image links and uploading images to the repository * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update docs/deploying-airbyte/on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update on-oci-vm.md * Add files via upload * Update on-oci-vm.md * Add files via upload * Update on-oci-vm.md * Update on-oci-vm.md Co-authored-by: Abhi Vaidyanatha <[email protected]> * 🐛 platform: Fix silent failures in sources (#4617) * add oracle dpeloyment guide to summary.md (#4619) * Mailchimp fix url-base (#4621) * minimal change to show acceptance test failure * exactly fix * bump version and readme * upd * 🎉 New Source: Paypal Transaction (#4240) * Added spec.json * Initialization * added oauth2 autorization * added spec, check, discover + catalogs/configurared_catalogs * updated request_params * added paging, slicing (1d) * Use oath2 for paypal * incremental sync, acceptance test * incremental sync, acceptance test * Added spec.json * Initialization * added oauth2 autorization * added spec, check, discover + catalogs/configurared_catalogs * updated request_params * added paging, slicing (1d) * Use oath2 for paypal * incremental sync, acceptance test * incremental sync, acceptance test * Added spec.json * Initialization * added oauth2 autorization * added spec, check, discover + catalogs/configurared_catalogs * updated request_params * added paging, slicing (1d) * Use oath2 for paypal * incremental sync, acceptance test * updated slices and api limits, added validation for input dates * added tests, fixed cursor related information in schemas and configured catalogs, removed old comments, re-arranged Base PaypalTransactionStream class * added input param 'env' to support production and sandbox envs * added support for sandbox option, updated pattern for optional end date option * added github secrets * added support for sandbox option, updated pattern for optional end date option * fixed Copyright date, removed debug mesages * added docs * fix for test failure - The sync should produce at least one STATE message * removed optional parameter 'end_date' * removed detailed info about balances schema * Delete employees.json * Delete customers.json * Added requests_per_minute rate limit * added unit tests, added custom backoff * added test for stream slices with stream state * removed comments * updated docs pages * fixed format for json files * fixed types in schemas and link to the schema. fixed primary key for Transactions stream * updated stream slices * Updated tests, unified stream_slices for both streams, all instance variables instantiated directly in __init__ method * added CHANGELOG.md * Added build seeds * fixed closing double quotation mark * added paypal entry in builds.md * add fixture helper * added paypal transaction generator script * fixed styling * maximum allowed start_date is extracted from API response now. * fixed schemas * fixed schemas - removed datetime * now maximum_allowed_start_date is identified by last_refreshed_datetime attr in API response. * added possibility to specify additional properties Co-authored-by: Sherif Nada <[email protected]> * set db version after full import is complete (#4626) * set db version after full import is complete * check db version in the last step * add comment * Fix docs formatting * Redirect old link to upgrading tutorial (#4635) Co-authored-by: Abhi Vaidyanatha <[email protected]> * Fix broken link in SUMMARY.md * Airflow Demo: Remove superset in down.sh (#4638) * Remove superset in down.sh * Clean up superset containers before creating them in up.sh Co-authored-by: Abhi Vaidyanatha <[email protected]> * Airflow demo: Clean up scripts and more clearly describe actions (#4639) * Airflow demo: Script cleanup * Correct docker compose name for airflow file * Final fixes * Clean up airbyte destination Co-authored-by: Abhi Vaidyanatha <[email protected]> * 🎉 Add documentation for configuring Kube GCS logging. (#4622) * Bump version: 0.27.0-alpha → 0.27.1-alpha (#4640) * 0.27.1 Platform Patch Notes (#4644) Co-authored-by: Abhi Vaidyanatha <[email protected]> * 🎉 New Source: Zendesk Sunshine (#4359) * pre-PR * add git config * format * Update airbyte-integrations/connectors/source-zendesk-sunshine/requirements.txt upd requirements.txt remove extra Co-authored-by: Eugene Kulak <[email protected]> * Update airbyte-integrations/connectors/source-zendesk-sunshine/source_zendesk_sunshine/streams.py backoff time int to float (btw real return type in headers is integer) Co-authored-by: Eugene Kulak <[email protected]> * requested changes * fix newline absence && rm unnecessary temp file * url_base to property * rm extra var coming property * rm extra var coming property * save * finishing updating the documentation * forgotten definition * add nullable to pass the test * fix date in the log Co-authored-by: Eugene Kulak <[email protected]> * 0.27.1 Connector Patch Notes (#4646) Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update connector certification table. (#4647) Co-authored-by: Abhi Vaidyanatha <[email protected]> * 🐛 Stub out the GCP Env Var in Docker to prevent noisy and harmless errors. (#4642) * Add this to prevent noisy errors. * Add hint to Airflow guide about local example (#4656) Co-authored-by: Abhi Vaidyanatha <[email protected]> * fix version for kube automatic migration support (#4649) * format zendesk sunshine connector (#4658) * 🎉 New source: Dixa (#4358) * Turn on MYSQL normalization flag. (#4651) * Turn on normalization flag. Bump versions * Combine admin and settings (#4525) * Add side menu component * Add side menu to settings page. Remove admin link from sidebar * Move NotificationPage * Move ConfigurationPage * Add Sources and Destinations pages to Settings. Delete Admin page * Add MetricsPage * Edit Notifications and Metrics pages * Update feedback for metrics and notification pages * Add update icons data to side menu * Add AccountPage * Job history purging (#4575) * WIP: Job history purging * Created test cases that handle variations of job history purging configuration * Typo fix * Expanded test cases to control for job history on multiple connections at once. * Handle latest job with saved state correctly regardless of order of ids * Whitespace * Externalized sql. Cleaned up constants. * Cleaned up test case persistence code and structure * Whitespace and formatting per standard tooling. * 0.27.1 Announcement Summary (#4678) Co-authored-by: Abhi Vaidyanatha <[email protected]> * 🐛 Source Sendgrid: add start_time config and correct primary_key (#4682) * add start_time config and correct primary_key * correct integration tests * correct type * config txt and primary_key * test to show how automatic migration handles deprecated definitions (#4655) * test to show definitions not present in latest seed would be deleted in automatic migration * format * add deprecated config being used scenario * Source dixa: fix unit tests (#4690) * introduce common abstraction for CDC via debezium (#4580) * wip * add file * final structure * few more updates * undo unwanted changes * add abstract test + more refinement * remove CDC metadata to debezium * rename class + add missing property * move debezium to bases + upgrade debezium version + review comments * downgrade version + minor fixes * reset to minutes * fix build * address review comments * should return Optional * use common abstraction for CDC via debezium for mysql (#4604) * use new cdc abstraction for mysql * undo wanted change * pull in latest changes * use renamed class + move constants to MySqlSource * bring in latest changes from cdc abstraction * format * bring in latest changes * pull in latest changes * use common abstraction for CDC via debezium for postgres (#4607) * use cdc abstraction for postgres * add files * ready * use renamed class + move constants to PostgresSource * bring in the latest changes * bring in latest changes * pull in latest changes * Source Dixa: Pin tz in ConversationExport.ms_timestamp_to_datetime (#4696) * Source Dixa: add to connector index (#4701) * allow injecting filters for server (#4677) * allow injecting filters * fmt * upgrade postgres version for new cdc abstraction (#4702) * Fix dependencies for Superset demo (#4705) * Fix superset dependency location * Add some Superset setup Co-authored-by: Abhi Vaidyanatha <[email protected]> * 📚 add SSH instructions for OCI VM setup (#4684) Co-authored-by: Sherif A. Nada <[email protected]> * upgrade mysql version for new cdc abstraction (#4703) * Update with ALTER TABLE statements (#4707) Co-authored-by: Abhi Vaidyanatha <[email protected]> * remove unused deps (#4512) Co-authored-by: Davin Chia <[email protected]> * fix config init race condition (#4679) * 🐛 Destination S3: fix minio output for parquet format * Bump destination s3 version (#4718) * Fix scheduler race condition. (#4691) * Periodic connector tests workflow: add `Accept` header per github docs recommendation (#4722) * allow launching integration tests from workflow dispatch (#4723) * Bump version: 0.27.1-alpha → 0.27.2-alpha (#4724) * 🐛 Source Square: Update _send_request method due to changes in Airbyte CDK (#4645) * 🎉 Destination Snowflake: tag snowflake traffic with airbyte ID to enable optimizations from Snowflake (#4713) * 🎉 New source: Typeform (#4541) Typeform source: Forms and Responses streams * Upgrade postgres and redshift destination to remove basic_normalization attribute (#4725) * upgrade snowflake,redshift,postgres to remove basic_normalization * undo snowflake * undo snowflaketest * fix broken assertions for automatic migration tests (#4732) * Slightly improve sed-based yaml parsing (#4721) Previous sed did not handle the valid `profile: foo` * throw exception if we close engine before snapshot is complete + increase timeout for subsequent records (#4730) * throw exception if we close engine before snapshot is complete + increase timeout for subsequent records * add comment + bump postgres version to use new changes * allow publishing airbyte-server to local maven repo (#4717) * allow publishing airbyte-server to local maven repo * Stub this out so the name that is created is airbyte-server-0.27.1-alpha.jar and not airbyte-server-0.27.1-alpha-all.jar. * Add comments. * see if this fixes build Co-authored-by: Davin Chia <[email protected]> * CDK: Add initial Destination abstraction and tests (#4719) Co-authored-by: Eugene Kulak <[email protected]> * Update docs on GitHub connector now that its Airbyte native (#4739) Co-authored-by: Abhi Vaidyanatha <[email protected]> * Remove statement about Postgres connector being based on Singer (#4740) Co-authored-by: Abhi Vaidyanatha <[email protected]> * fix flaky migration acceptance test (#4743) * upgrade fabric8 client (#4738) * 🎉 Source MSSQL: implementation for CDC (#4689) * first few classes for mssql cdc * wip * mssql cdc working against unit tests * increment version * add cdc acceptance test * tweaks * add file * working on comprehensive tests * change isolation from snapshot to read_committed_snapshot * finalised type tests * Revert "change isolation from snapshot to read_committed_snapshot" This reverts commit 20c6768. * small docstring fix * remove unused imports * stress test fixes * minor formatting improvements * mssql cdc docs * finish off cdc docs * format fix * update connector version * add to changelog * fix for sql server agent offline failing cdc enable on tables * final structure * few more updates * undo unwanted changes * add abstract test + more refinement * remove CDC metadata to debezium * use new cdc abstraction for mysql * undo wanted change * use cdc abstraction for postgres * add files * pull in latest changes * ready * rename class + add missing property * use renamed class + move constants to MySqlSource * use renamed class + move constants to PostgresSource * move debezium to bases + upgrade debezium version + review comments * downgrade version + minor fixes * bring in latest changes from cdc abstraction * reset to minutes * bring in the latest changes * format * fix build * address review comments * bring in latest changes * bring in latest changes * use common abstraction for CDC via debezium for sql server * remove debezium from build * finalise PR * should return Optional * pull in latest changes * pull in latest changes * address review comments * use common abstraction for CDC via debezium for mysql (#4604) * use new cdc abstraction for mysql * undo wanted change * pull in latest changes * use renamed class + move constants to MySqlSource * bring in latest changes from cdc abstraction * format * bring in latest changes * pull in latest changes * use common abstraction for CDC via debezium for postgres (#4607) * use cdc abstraction for postgres * add files * ready * use renamed class + move constants to PostgresSource * bring in the latest changes * bring in latest changes * pull in latest changes * lower version for tests to run on CI * format * Update docs/integrations/sources/mssql.md Co-authored-by: Sherif A. Nada <[email protected]> * addressing review comments * fix for testGetTargetPosition * format changes Co-authored-by: George Claireaux <[email protected]> Co-authored-by: Sherif A. Nada <[email protected]> * bump up MSSQL version for cdc (#4694) * first few classes for mssql cdc * wip * mssql cdc working against unit tests * increment version * add cdc acceptance test * tweaks * add file * working on comprehensive tests * change isolation from snapshot to read_committed_snapshot * finalised type tests * Revert "change isolation from snapshot to read_committed_snapshot" This reverts commit 20c6768. * small docstring fix * remove unused imports * stress test fixes * minor formatting improvements * mssql cdc docs * finish off cdc docs * format fix * update connector version * add to changelog * fix for sql server agent offline failing cdc enable on tables * final structure * few more updates * undo unwanted changes * add abstract test + more refinement * remove CDC metadata to debezium * use new cdc abstraction for mysql * undo wanted change * use cdc abstraction for postgres * add files * pull in latest changes * ready * rename class + add missing property * use renamed class + move constants to MySqlSource * use renamed class + move constants to PostgresSource * move debezium to bases + upgrade debezium version + review comments * downgrade version + minor fixes * bring in latest changes from cdc abstraction * reset to minutes * bring in the latest changes * format * fix build * address review comments * bring in latest changes * bring in latest changes * use common abstraction for CDC via debezium for sql server * remove debezium from build * finalise PR * should return Optional * pull in latest changes * pull in latest changes * address review comments * use common abstraction for CDC via debezium for mysql (#4604) * use new cdc abstraction for mysql * undo wanted change * pull in latest changes * use renamed class + move constants to MySqlSource * bring in latest changes from cdc abstraction * format * bring in latest changes * pull in latest changes * use common abstraction for CDC via debezium for postgres (#4607) * use cdc abstraction for postgres * add files * ready * use renamed class + move constants to PostgresSource * bring in the latest changes * bring in latest changes * pull in latest changes * lower version for tests to run on CI * bump up mssql version for cdc * format * Update docs/integrations/sources/mssql.md Co-authored-by: Sherif A. Nada <[email protected]> * addressing review comments * fix for testGetTargetPosition * format changes Co-authored-by: George Claireaux <[email protected]> Co-authored-by: Sherif A. Nada <[email protected]> * fixed broken links and styling (#4747) * Fix enabling connection in refresh catalog mode (#4527) * Fix enabling connection in refresh catalog mode * Do not update deprecated connectors (#4674) * Do not update deprecated connectors * Fix various connectorDefinition issues: disappearing button, wrong id used for destination update * 🐛 Source Slack: add float_ts field (#4683) * rename float_ts to ts cursor_field * add float_ts * change float_ts to number * change channel_msg * bump version * increase default timeout_seconds slack acc test * timeout_seconds to 1750 * timeout_seconds to 3600 :p * add changelog for slack connector * copy docs to webapp docker image (#4522) * use kube service user for pod sweeper (#4737) * use kube service user for pod sweeper * add pod sweeper logs * temporarily switch to stable for testing * temporarily remove building steps for kube testing since it can use prod images * output date strings from date command * load stable images * remove loading since it can pull the images * increase window for success storage to two hours * revert test logging changes * 🐛 Source GitHub: fix bug with `IssueEvents` stream and add handling for rate limiting (#4708) * Few updates for GitHub source Set correct `cursor_field` for `IssueEvents` stream. Add rate limit handling. Add handling for 403 error. Add handling for 502 error. Co-authored-by: Eugene Kulak <[email protected]> Co-authored-by: Sherif A. Nada <[email protected]> * 🐛 Fix some api-spec errors. (#4742) * Source PostHog: Use account information for checking the connection (#4692) * this should fix the check if no records in annotations stream * update schemas for new SAT requirements && apply user hint upgrade on wrong api key * save schema upd * upd insights schema * upd insights schema2 * upd insights schema3 * upd insights schema4 * upd insights schema5 (null is joking) * upd insights schema6 (null is joking) * upd insights schema7 * upd insights schema8 * upd insights schema8 * bump version && docs * SAT: Improve error message when data mismatches schema (#4753) * improve message when data mismatch schema Co-authored-by: Eugene Kulak <[email protected]> * increase sleep duration + show logs in CI (#4756) * Fixed cockroachdb repo image (#4758) * Bump version: 0.27.2-alpha → 0.27.3-alpha (#4761) * update kube docs (#4749) * fix kube overlay version (#4765) * Split Platform and Connector Builds (#4514) * remove second docs check in build(#4766) * Restore template generator and fix formatting. (#4768) * connector generate: fix chown logic (#4774) * Remove example use cases from docs (#4775) Co-authored-by: Abhi Vaidyanatha <[email protected]> * Update README.md * 🎉 All java connectors: Added configValidator to check, discover, read and write calls (#4699) * Added configValidator to java connectors * 🎉 Stripe Source: Fix subscriptions stream to return all kinds of subscriptions (including expired and canceled) (#4669) #4669 Stripe Source: Fix subscriptions stream to return all kinds of subscriptions (including expired and canceled) Co-authored-by: Oleksandr Bazarnov <[email protected]> * Add note about orphaned Airbyte configs preventing automatic upgrades (#4709) * Add note about removing orphaned Airbyte configs * Remove excess baggage * Add a resetting section to make this more clear. Co-authored-by: Abhi Vaidyanatha <[email protected]> * Patch 0.27.2 and 0.27.3 platform notes (#4792) Co-authored-by: Abhi Vaidyanatha <[email protected]> * Connector notes for 0.27.3 (#4794) Co-authored-by: Abhi Vaidyanatha <[email protected]> * Add new logo to GitHub page (#4796) Co-authored-by: Abhi Vaidyanatha <[email protected]> * 🎉 New Destination: Google Cloud Storage (#4784) * Adding Google Cloud Storage as destination * Removed few comments and amended the version * Added documentation in docs/integrations/destinations/gcs.md * Amended gcs.md with the right pull id * Implemented all the fixes requested by tuliren as per #4329 * Renaming all the files * Branch alligned to S3 0.1.7 (with Avro and Jsonl). Removed redundant file by making S3 a dependency for GCS * Removed some additional duplicates between GCS and S3 * Revert changes in the root files * Revert jdbc files * Fix package names * Refactor gcs config * Format code * Fix gcs connection * Format code * Add acceptance tests * Fix parquet acceptance test * Add ci credentials * Register the connector and update documentations * Fix typo * Format code * Add unit test * Add comments * Update readme Co-authored-by: Sherif A. Nada <[email protected]> Co-authored-by: Marco Fontana <[email protected]> Co-authored-by: [email protected] <[email protected]> Co-authored-by: Marco Fontana <[email protected]> Co-authored-by: Sherif A. Nada <[email protected]> * 🐛 CDK: Fix logging of initial state value (#4795) * Update abstract_source.py * bump * CHANGELOG.md Co-authored-by: Eugene Kulak <[email protected]> * bug fix: use register api (#4811) * 🐛 Add missing dependencies for acceptance tests to run. (#4808) * 🎉 Add Python Destination Template (#4771) * Format. (#4814) * 🎉 Migrate config persistence to database (#4670) * Implement db config persistence * Fix database readiness check * Reduce logging noise * Setup config database in config persistence factory * Update documentation * Load seed from yaml files * Refactor config persistence factory * Add one more test to mimic migration * Remove unnecessary changes * Run code formatter * Update placeholder env values * Set default config database parameters in docker compose Co-authored-by: Christophe Duong <[email protected]> * Default setupDatabase to false * Rename variable * Set default config db parameters for server * Remove config db parameters from the env file * Remove unnecessary environment statements * Hide config persistence factory (#4772) * Remove CONFIG_DATABASE_HOST * Use builder in the test * Simplify config persistence builder * Clarify config db connection readiness * Format code * Add logging * Fix typo Co-authored-by: Christophe Duong <[email protected]> * Add a config_id only index * Reuse record insertion code * Add id field name to config schema * Support data loading from legacy config schemas * Log missing logs in migration test * Move airbyte configs table to separate directory * Update exception message * Dump specific tables from the job database * Remove postgres specific uuid extension * Comment out future branch * Default configs db variables to empty When defaulting them to the jobs db variables, it somehow does not work. * Log inserted config records * Log all db write operations * Add back config db variables in env file to mute warnings * Log connection exception to debug flaky e2e test * Leave config db variables empty `.env` file does not support variable expansion. Co-authored-by: Christophe Duong <[email protected]> Co-authored-by: Charles <[email protected]> * 🎉 Source intercom: migration to CDK (#4676) * Added Intercom implementation * Updated segments docs * Updated _send_request method to new airbyte-cdk version * Updated cursor field to datetime string * Added filtering by state for incremental sync * Updated cursor paths for test incremental sync * Added dict type validation to get_data method * Updated catalog * Updated typing for start_date * Updated singer seed to cdk seed * Updated connector docs * Updated sample config file * Sorted streams alphabetically * Removed placeholder comments * Renamed rate_limit to queries_per_hour * Updated common sleep time to backoff_time method * 🎉 New source: Pipedrive connector (#4686) * Add pipedrive source initial * Add initial schemas. Add MVP source implementation. * Implement MVP streams * Complete MVP streams implementation * Apply schema format * Add test creds * Update streams.py Fix schemas * Update replication_start_date format. Add extra pagination condition * Refactor streams, remove unused classes. * Add pipedrive.md docs file. Add Pipedrive source definitions. * Add json source definition. * Update spec.json * Add docs mentions throughout the project files * Make number of Concurrent Jobs configurable. (#4687) * Explicitly pin ec2 runner version to 2.2.1. (#4823) This was a mismash before, partially my fault. Explicitly pinning for now. * 🐛 Source Facebook: Improve rate limit management (#4820) * Improve rate limit management * bump version * facebook-marketing.md update the changelog * format and fix * Source Facebook: fix formatting and publish new version (#4826) * format * disable schema validation * fix urls in AdCreatives stream, enable SAT for creatives * format Co-authored-by: Eugene Kulak <[email protected]> * Code generator: Update generator to chown docs and config definition directories (#4819) * Python Demo Destination: KVDB (#4786) * 📚 CDK: Add python destination tutorial (#4800) * 📚 Source Shopify: migrate to new sandbox, update API version to 2021-07 (#4830) (#4830) Source Shopify: migrate to new sandbox, update API version to 2021-07 Co-authored-by: Oleksandr Bazarnov <[email protected]> * 🐛 Source Instagram: Read previous state format and upgrade it (#4805) * few fixes for user_insights state * support old state format * format * bump Co-authored-by: Eugene Kulak <[email protected]> * Add placeholder (#4816) * Add update button (#4809) * Point to new location for connector build status history (#4840) * Update GAds docs to indicate incremental support * Add openreplay (#4685) * Add openreplay * Add env variables for openreplay * Add openreplay env for k8s * 🎉 Source mixpanel: migration to CDK (#4566) * Mixpanel initiation * copied schemas and specs file from singer connector * authentication and a few streams * Added Funnels + FunnelsList * Added example of funnel response * added incremental Funnels stream with tests * added Annotations, CohortMembers, Engage, Cohorts, Funnels * added Revenue * fixed formatting * fixed variable names * fixed cohort_members and updated export streams * moved start_date and date checks into SourceMixpanel class * added error handling * added unit test, update docs and ci creds * fix url base for export stream * added full and incremental read for export stream * updated acceptance tests, added limit correction based on number of streams, export cursor is stored in datatime string * Funnel stream - added complex state which contains state for each funnel * added attribution windows support and project timezone config * fixed formatting * added default timezone * added dynamic schema generation for Engage and Export streams * fixed formatting * fixed ability to pass start_date in datetime format as well * fixed ability to pass start_date in datetime format as well * added additional_properties field for dynamic schemas. updates regex for start_date matching to support old config file * fixed formatting * export stream - convert all values to default type - string * added schema ref * added new properties for funnel stream * fixed formatting in funnel schema * added build related files * update changelog * fixed and added comments, renamed rate_limit variable * fixed formatting * changed normalization for reserved mixpanel attributes like $browser * alphabetise spec fields * added description about API limit handling * updated comment * Add openreplay variable (#4844) * 🐛 Sendgrid source: Gracefully handle malformed responses from sendgrid API (#4839) * Update job description (#4848) * Update job description * Create senior-product-manager * Create founding-account-executive * Update senior-product-manager * Update SUMMARY.md * Add py destination tutorial to summary.md (#4853) * Update CHANGELOG.md * 🐛 Kube: Fix Source Ports not releasing. (#4822) Closes #4660 . On further investigation, it turns out we were not releasing the source ports. This is because of how the Process abstraction works - waitFor calls close under the hood. We were only calling waitFor if the process was still alive. This is determined by the exitValue which comes from the Kubernetes pod's termination status. However, these ports are a local resource and no close calls means they were left dangling, leading to the behaviour we see today. Explicitly call close after retrieving the exit value of the Kubernetes pod. This better follows traditional assumptions around Processes - if the process returns some exit value, it means all resources associated with that process have been cleaned up. Also, - add in a bunch of debug logging for the future. - have better names for Kubernetes workers to make operations easier. * use new AMI ID for connector builds (#4855) * Wait for config volume to be ready (#4835) * Do not create config directory in fs persistence construction * Run kube acceptance test only for testing purpose * Wait for config volume to be ready * Move config volume wait for fs persistence construction * Restore ci workflow * Prune imports * 🎉 New source: US census (#4228) Co-authored-by: Sherif Nada <[email protected]> * publish US Census (connector) (#4857) Co-authored-by: Daniel Mateus Pires <[email protected]> Co-authored-by: Daniel Mateus Pires <[email protected]> * 🐛 Source JIRA: Fix DBT failing normalization on `Labels` schema. (#4817) (#4817) 🐛 Source JIRA: Fix DBT failing normalization on `Labels` schema. Co-authored-by: Oleksandr Bazarnov <[email protected]> * Rename founding-account-executive to founding-account-executive.md * Tweak ConfigNotFoundException class (#4821) * Use internal_api_host env variable * Source ZenDesk: format and validate code * refactor import / export endpoints to use the same code path as auto migration (#4797) * fix build (#4865) * 📝 Add server version requirement for mysql normalization (#4856) * 🐛 Destination MySQL: fix problem if source has a column with json (#4825) * [4583] Fixed MySQL destination of fails is source has a column with json data * hotfix: rename senior PM file to add .md * 📚 improve mongo docs and param descriptions (#4870) * Remove duplicated seed repository (#4869) * add workspace helper (#4868) * add workspace helper * fmt * switch to a fixed limit * 🐛 Fix Oracle spec to declare `sid` instead of `database` param, Redshift to allow `additionalProperties`, MSSQL test and spec to declare spec type correctly (#4874) * Kube: Better Port Abstraction. (#4829) Introduce a better port abstraction whose primary purpose is to confirm that ports are released when the Kube Pod Process is closed. This prevents issues like #4660 I'm also opening more ports so we can run at least 10 syncs in parallel. * Source Zendesk: update docs * Remove unused files * add a stream_slices logic for ticket_comments stream * remove changes of other connections * add secret Zendesk keys to command configs * 🐛 Source Zendesk Support: add dummy unit test * add dummy integration test * fix Zendesk not loading username and facebook/twitter id #4373 * sort streams alphabetically * fix test issue with the unsupport field validate_output_from_all_streams * add info to source_definitions.yaml * remove json_schema from configured_catalog.json * add backoff logic * add unit tests * move part of unit tests to integration tests * fix test dependencies * add a build status Co-authored-by: Maksym Pavlenok <[email protected]> Co-authored-by: Sherif A. Nada <[email protected]> Co-authored-by: LiRen Tu <[email protected]> Co-authored-by: vovavovavovavova <[email protected]> Co-authored-by: Eugene Kulak <[email protected]> Co-authored-by: Abhi Vaidyanatha <[email protected]> Co-authored-by: Abhi Vaidyanatha <[email protected]> Co-authored-by: Shadab Mohammad <[email protected]> Co-authored-by: midavadim <[email protected]> Co-authored-by: Subodh Kant Chaturvedi <[email protected]> Co-authored-by: Davin Chia <[email protected]> Co-authored-by: Oliver Meyer <[email protected]> Co-authored-by: Artem Astapenko <[email protected]> Co-authored-by: Jenny Brown <[email protected]> Co-authored-by: Marcos Marx <[email protected]> Co-authored-by: Jared Rhizor <[email protected]> Co-authored-by: Charles <[email protected]> Co-authored-by: Varun B Patil <[email protected]> Co-authored-by: Dmytro <[email protected]> Co-authored-by: Yaroslav Dudar <[email protected]> Co-authored-by: Brian Krausz <[email protected]> Co-authored-by: George Claireaux <[email protected]> Co-authored-by: oleh.zorenko <[email protected]> Co-authored-by: Eugene Kulak <[email protected]> Co-authored-by: Eugene <[email protected]> Co-authored-by: John Lafleur <[email protected]> Co-authored-by: Anna Lvova <[email protected]> Co-authored-by: Marco Fontana <[email protected]> Co-authored-by: [email protected] <[email protected]> Co-authored-by: Marco Fontana <[email protected]> Co-authored-by: Christophe Duong <[email protected]> Co-authored-by: Serhii Lazebnyi <[email protected]> Co-authored-by: Vadym <[email protected]> Co-authored-by: Vladimir remar <[email protected]> Co-authored-by: Oleksandr <[email protected]> Co-authored-by: Oleksandr Bazarnov <[email protected]> Co-authored-by: Daniel Mateus Pires <[email protected]> Co-authored-by: Daniel Mateus Pires <[email protected]> Co-authored-by: jrhizor <[email protected]>

sherifnada added 2 commits July 2, 2021 00:20

allow sending arbitrary kwargs on a request

e0178bd

fmt

dd8f924

github-actions bot added the CDK Connector Development Kit label Jul 2, 2021

sherifnada added 2 commits July 2, 2021 13:28

Update http.py

e0e5817

Update http.py

39b1c12

sherifnada changed the title ~~CDK: Allow setting kwargs on outgoing HTTP requests~~ CDK: Allow setting adapter kwargs on outgoing HTTP requests Jul 2, 2021

Update CHANGELOG.md

bf39c43

sherifnada requested review from davinchia, keu and tuliren July 2, 2021 20:32

sherifnada added 2 commits July 2, 2021 13:52

Merge branch 'master' of github.com:airbytehq/airbyte into sherif/exp…

1bc4bb1

…ose-kwargs-in-http-stream

Merge branch 'sherif/expose-kwargs-in-http-stream' of github.com:airb…

bbc6d7e

…ytehq/airbyte into sherif/expose-kwargs-in-http-stream

kill build cache

d021dd4

sherifnada commented Jul 2, 2021

View reviewed changes

Merge branch 'master' of github.com:airbytehq/airbyte into sherif/exp…

df5d4bf

…ose-kwargs-in-http-stream

davinchia approved these changes Jul 3, 2021

View reviewed changes

Merge branch 'master' of github.com:airbytehq/airbyte into sherif/exp…

72663c5

…ose-kwargs-in-http-stream

keu reviewed Jul 4, 2021

View reviewed changes

keu approved these changes Jul 4, 2021

View reviewed changes

sherifnada changed the title ~~CDK: Allow setting adapter kwargs on outgoing HTTP requests~~ 🎉 Python CDK: Allow setting network adapter kwargs on outgoing HTTP requests Jul 7, 2021

sherifnada changed the title ~~🎉 Python CDK: Allow setting network adapter kwargs on outgoing HTTP requests~~ 🎉 Python CDK: Allow setting network adapter args on outgoing HTTP requests Jul 7, 2021

sherifnada added 2 commits July 7, 2021 10:39

Merge branch 'master' of github.com:airbytehq/airbyte into sherif/exp…

332f61d

…ose-kwargs-in-http-stream

docs

e5710c5

sherifnada merged commit 3aebe55 into master Jul 7, 2021

sherifnada deleted the sherif/expose-kwargs-in-http-stream branch July 7, 2021 22:18

antixar pushed a commit that referenced this pull request Jul 22, 2021

🎉 Python CDK: Allow setting network adapter args on outgoing HTTP req…

f02d74a

…uests (#4493)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🎉 Python CDK: Allow setting network adapter args on outgoing HTTP requests #4493

🎉 Python CDK: Allow setting network adapter args on outgoing HTTP requests #4493

sherifnada commented Jul 2, 2021

sherifnada commented Jul 2, 2021 •

edited by github-actions bot

Loading

sherifnada commented Jul 2, 2021 •

edited by github-actions bot

Loading

sherifnada commented Jul 2, 2021 •

edited by github-actions bot

Loading

sherifnada Jul 2, 2021

davinchia left a comment

sherifnada commented Jul 4, 2021 •

edited by github-actions bot

Loading

keu Jul 4, 2021

keu Jul 4, 2021

keu Jul 4, 2021

keu Jul 4, 2021 •

edited

Loading

keu Jul 4, 2021

sherifnada Jul 7, 2021

keu left a comment

sherifnada commented Jul 7, 2021 •

edited by github-actions bot

Loading

sherifnada commented Jul 7, 2021 •

edited by github-actions bot

Loading

🎉 Python CDK: Allow setting network adapter args on outgoing HTTP requests #4493

🎉 Python CDK: Allow setting network adapter args on outgoing HTTP requests #4493

Conversation

sherifnada commented Jul 2, 2021

What

How

sherifnada commented Jul 2, 2021 • edited by github-actions bot Loading

sherifnada commented Jul 2, 2021 • edited by github-actions bot Loading

sherifnada commented Jul 2, 2021 • edited by github-actions bot Loading

sherifnada Jul 2, 2021

Choose a reason for hiding this comment

davinchia left a comment

Choose a reason for hiding this comment

sherifnada commented Jul 4, 2021 • edited by github-actions bot Loading

keu Jul 4, 2021

Choose a reason for hiding this comment

keu Jul 4, 2021

Choose a reason for hiding this comment

keu Jul 4, 2021

Choose a reason for hiding this comment

keu Jul 4, 2021 • edited Loading

Choose a reason for hiding this comment

keu Jul 4, 2021

Choose a reason for hiding this comment

sherifnada Jul 7, 2021

Choose a reason for hiding this comment

keu left a comment

Choose a reason for hiding this comment

sherifnada commented Jul 7, 2021 • edited by github-actions bot Loading

sherifnada commented Jul 7, 2021 • edited by github-actions bot Loading

sherifnada commented Jul 2, 2021 •

edited by github-actions bot

Loading

sherifnada commented Jul 2, 2021 •

edited by github-actions bot

Loading

sherifnada commented Jul 2, 2021 •

edited by github-actions bot

Loading

sherifnada commented Jul 4, 2021 •

edited by github-actions bot

Loading

keu Jul 4, 2021 •

edited

Loading

sherifnada commented Jul 7, 2021 •

edited by github-actions bot

Loading

sherifnada commented Jul 7, 2021 •

edited by github-actions bot

Loading