Source Recurly: adds state_checkpoint_interval to streams
#13685
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
marcosmarxm
approved these changes
Jun 10, 2022
github-actions
bot
added
the
area/documentation
Thanks @marcosmarxm for the review. Bumped the version of the Recurly connector to |
|
/test connector=connectors/source-recurly
Build FailedTest summary info: |
|
Sorry @mohamagdy havving problems with our CI. |
|
Our CI is failing because isn't possible to implement retry policy using current implementation. |
|
@mohamagdy I'm blocked to merge this waiting https://github.com/airbytehq/airbyte-internal-issues/issues/751. I'll be on vacation and only return 26-jul maybe after I'll have some time to fix that |
|
/test connector=connectors/source-recurly
Build FailedTest summary info: |
marcosmarxm
force-pushed
the
add-state-checkpoint-to-recurly-stream
branch
from
d8c462b to
910133b
Compare
|
/publish connector=connectors/source-recurly run-tests=false
if you have connectors that successfully published but failed definition generation, follow step 4 here |
marcosmarxm
changed the title
state_checkpoint_interval to Recurly streamstate_checkpoint_interval to streams
girarda
pushed a commit
that referenced
this pull request
Aug 11, 2022
* Add `state_checkpoint_interval` to Recurly stream * Bumpg Recurly source version to `0.4.1`
girarda
added a commit
that referenced
this pull request
Aug 12, 2022
* greenhouse minus pagination * jobs * first substream * rename field * applications_demographics_answers_stream * interviews * All streams are implemented * fix check * fix spec * disable backward compatibility tests * disable backward compatibility tests * unit tests * definitions * only use config.json * bump version * expected records * delete stream classes * Handle extracting no records from root * handle missing keys * Remove unused field from JsonSchema (#15425) * few fixes from working with sendgrid * reset to master * only update the docstring * reset * 🎉 Source File - add support for custom encoding (#15293) * added support for custom encoding * fixed unit test for utf16 * updated docs * bumped connector version * auto-bump connector version [ci skip] Co-authored-by: Octavia Squidington III <[email protected]> * change query frequency to 1hour (#15499) * [low-code connectors]: Assert there are no custom top-level fields (#15489) * move components to definitions field * Also update the references * validate the top level fields and add version * raise exception on unknown fields * newline * unit tests * set version to 0.1.0 * newline * 🐞 Postgres source: fix bug in intermediate state emission (#15496) * Rename record counter * Rename method * Emit intermediate state after all cursor records * Emit intermediate state only when it is ready * Merge two checks * Add a testing message * Fix unit tests * Add one more testing record and add comments * Add test case for multiple records with the same cursor value * Revert irrelevant change * Add explanation in javadoc * Format code * Rename testing methods * Fix comment * Bump version * auto-bump connector version [ci skip] Co-authored-by: Octavia Squidington III <[email protected]> * 🪟 🔧 Add testing and storybook component for CatalogDiffModal (#15426) * wip diff modal test setup * starting storybook add * storybook working now * cleanup * aria labels * test syncmode string * 🎉 New Source: Hubplanner (#15521) * added hubplanner source connector * feat: added more streams to read from * cleaned up some unneeded integration tests * fix hubplanner schema * changes * update dockerfile * add seed and doc * update spec * run source spec seed file Co-authored-by: Ricky Renner <[email protected]> * Make it possible to specify normalization pod resources. (#15495) Today we are running into OOM exceptions with normalization. Normalization itself also inherits the destination's resource requirements. After work to bring destination memory usage down, this is no longer ideal, since most destinations use less memory than normalization needs. This PR makes it possible to specify the general resource the normalization pod is provided via env vars. Notes: - Add env vars. Default to the various job main container resources if these are not set. - Instead of using the destination's memory, use the normalization specify env vars. * [low-code connectors] Extract datetime parser and handle %s format directive (#15429) * fix parse * Revert "fix parse" This reverts commit 3c76c5a. * fix parse timestamp * extract datetime parser * remove print * use parser * top level docstring * rename variable * do not use timestamp() * Revert "do not use timestamp()" This reverts commit 016cb69. * update comment * bump cdk version * Update template * source-file-secure bump to 0.2.16 (#15528) * update Dockerfile version * update init to accept additional args * unit test sendgrid messages stream (#15331) * unit test sendgrid messages stream * reset * Update airbyte-integrations/connectors/source-sendgrid/unit_tests/unit_test.py Co-authored-by: Augustin <[email protected]> Co-authored-by: Augustin <[email protected]> * record extractor interface * dpath extractor * docstring * 🎉 Source File: cache binary stream to file (#15501) Signed-off-by: Sergey Chvalyuk <[email protected]> * Docs: update posthog.md (#15541) * Source Stripe: implement slicing (#15292) * #45 oncall - source Stripe: implement slicing * #45 source stripe: upd changelog * #45 source stripe: upd changelog * #45 source stripe: make slice range configurable * #45 source stripe: move generating a single slice into a mixin * auto-bump connector version [ci skip] Co-authored-by: Octavia Squidington III <[email protected]> * fix: revert extraEnv delition in values.yaml for bootloader (#15548) * fix: revert extraEnv delition in values.yaml for bootloader * add newline * SAT: backward compatibility - check that cursor fields were not changed (#15520) * Replace twttr repo to the root build.gradle (#15544) * Fixed bucket naming for S3 * replaced twttr repo to the root build.gradle * replaced twttr repo to the root build.gradle Co-authored-by: Oleksandr Sheheda <[email protected]> * Generate separate server endpoints per domain (#15513) * 🐛 Backward compatibility test: Don't fail on updating additionalProperties (#15532) * Source Recurly: adds `state_checkpoint_interval` to streams (#13685) * Add `state_checkpoint_interval` to Recurly stream * Bumpg Recurly source version to `0.4.1` * reset * use dpath * enable backward compatibility test * infer types * Revert "infer types" This reverts commit b4de8d6. * infer some of the types * some drying * more drying * auto-bump connector version [ci skip] Signed-off-by: Sergey Chvalyuk <[email protected]> Co-authored-by: midavadim <[email protected]> Co-authored-by: Octavia Squidington III <[email protected]> Co-authored-by: Xiaohan Song <[email protected]> Co-authored-by: Liren Tu <[email protected]> Co-authored-by: Teal Larson <[email protected]> Co-authored-by: Marcos Marx <[email protected]> Co-authored-by: Ricky Renner <[email protected]> Co-authored-by: Davin Chia <[email protected]> Co-authored-by: Brian Lai <[email protected]> Co-authored-by: Augustin <[email protected]> Co-authored-by: Serhii Chvaliuk <[email protected]> Co-authored-by: juliatournant <[email protected]> Co-authored-by: Denys Davydov <[email protected]> Co-authored-by: Kyryl Skobylko <[email protected]> Co-authored-by: VitaliiMaltsev <[email protected]> Co-authored-by: Oleksandr Sheheda <[email protected]> Co-authored-by: Jonathan Pearlin <[email protected]> Co-authored-by: Mohamed Magdy <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What
Add
state_checkpoint_intervalto Recurly source connector🚨 User Impact 🚨
No user impact expected
Pre-merge Checklist
Updating a connector
Community member or Airbyter
airbyte_secret./gradlew :airbyte-integrations:connectors:<name>:integrationTest.README.mdbootstrap.md. See description and examplesdocs/integrations/<source or destination>/<name>.mdincluding changelog. See changelog exampleTests
Unit
airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_account_coupon_redemptions_read_records ✓ 5% ▌ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_account_notes_read_records ✓ 10% ▉ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_accounts_methods_client_method_name ✓ 14% █▌ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_add_ons_client_method_name ✓ 19% █▉ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_billing_infos_client_method_name ✓ 24% ██▍ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_coupons_methods_client_method_name ✓ 29% ██▉ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_credit_payments_read_records ✓ 33% ███▍ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_export_dates_read_records ✓ 38% ███▊ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_get_updated_state ✓ 43% ████▍ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_invoices_methods_client_method_name ✓ 48% ████▊ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_line_items_methods_client_method_name ✓ 52% █████▎ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_measured_unit_client_method_name ✓ 57% █████▊ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_plans_client_method_name ✓ 62% ██████▎ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_read_records ✓ 67% ██████▋ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_read_records_with_begin_time ✓ 71% ███████▎ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_read_records_with_end_time ✓ 76% ███████▋ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_shipping_addresses_client_method_name ✓ 81% ████████▏ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_shipping_methods_client_method_name ✓ 86% ████████▋ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_subscriptions_client_method_name ✓ 90% █████████▏ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_transactions_client_method_name ✓ 95% █████████▌ airbyte-integrations/connectors/source-recurly/unit_tests/test_streams.py::TestStreams.test_unique_coupons_read_records ✓ 100% ██████████ Results (0.69s): 21 passedIntegration