✨ Source Stripe: have all full_refresh stream syncs be concurrent

[maxi297]

What

Addresses #32308

How

Creating a set of all stream names that are full refresh from the catalog and if a stream name is within this set, create a stream using StreamFacade instead

Notes

There is an acceptance criteria that mentions Verify that the streams that do not have CAT coverage still work as expected. However I could not test some of those streams because:

• Could not find streams with records in the last 7 days
• Stream name was shared with other sources and I timeboxeds manually mapping from log to a stripe sync with records after 5 minutes for each stream

Streams I could not test:

• application_fees_refunds
• authorizations
• external_account_cards
• payment_methods
• persons
• reviews
• transactions

Note that testing for early_fraud_warnings yielded only one record.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
airbyte-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 14, 2023 2:32am

[github-actions]

Before Merging a Connector Pull Request

Wow! What a great pull request you have here! 🎉

To merge this PR, ensure the following has been done/considered for each connector added or updated:

• PR name follows PR naming conventions
• Breaking changes are considered. If a Breaking Change is being introduced, ensure an Airbyte engineer has created a Breaking Change Plan.
• Connector version has been incremented in the Dockerfile and metadata.yaml according to our Semantic Versioning for Connectors guidelines
• You've updated the connector's metadata.yaml file any other relevant changes, including a breakingChanges entry for major version bumps. See metadata.yaml docs
• Secrets in the connector's spec are annotated with airbyte_secret
• All documentation files are up to date. (README.md, bootstrap.md, docs.md, etc...)
• Changelog updated in docs/integrations/<source or destination>/<name>.md with an entry for the new version. See changelog example
• Migration guide updated in docs/integrations/<source or destination>/<name>-migrations.md with an entry for the new version, if the version is a breaking change. See migration guide example
• If set, you've ensured the icon is present in the platform-internal repo. (Docs)

If the checklist is complete, but the CI check is failing,

1. Check for hidden checklists in your PR description

2. Toggle the github label checklist-action-run on/off to re-run the checklist CI.

[maxi297]

It was like this before but this interface feels weird as I would expect:

source = SourceStripe(<a catalog with only one stream>
streams = source.streams()
assert len(streams) == 1

However, all streams are returned there. I guess otherwise would be a breaking change as streams has always returned everything. There are also cases where we don't have the catalog (for example, discover)

EDIT: would it help if we change the naming to concurrent_full_refresh_catalog? It seems very weird but many it's more aligned with what it does...

[girarda]

this is great!