github-actions[bot] closed pull request #37244: FIX: Update Airflow Airbyte
Provider to use the new Airbyte API
URL: https://github.com/apache/airflow/pull/37244
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
github-actions[bot] commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-2019139236
This pull request has been automatically marked as stale because it has not
had recent activity. It will be closed in 5 days if no further activity occurs.
Thank you for you
potiuk commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-1936468182
so it's ~ the threshold I had in mind. I'd say it's a bit too early to
remove it then and supporting both should be preferred
--
This is an automated message from the Apache Git Servic
Taragolis commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-1936305925
Just for the record, I've checked latest Airbyte `0.50.48` locally and both
APIs available. I'm not sure but last time I've found that presumably [v0.50.13
](https://github.com/airbyt
potiuk commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-1935951677
If it was a simple breaking change in parameters that users could correct in
their DAGs. I'd be for immediate breaking change. But the fact that users might
be using an older version of
Taragolis commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-1935915086
Sorry for delay response.
>What do you think about asking people to downgrade their Airflow Airbyte
In general it is an option, however always unknown how many users read
mitcheccles commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-1934200671
> **Option 1**
> If we would like to introduce Breaking Changes then we need to create
notes into the Change log and potentially simple migration guide, e.g. minimal
version of A
mitcheccles commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-1934009185
> > The changes in this PR do not use the official Python SDK. This SDK is
still in beta.
>
> Is there official statement of Airbyte saying that? I couldn't find such
stateme
mitcheccles commented on code in PR #37244:
URL: https://github.com/apache/airflow/pull/37244#discussion_r1482888530
##
airflow/providers/airbyte/__init__.py:
##
@@ -27,7 +27,7 @@
__all__ = ["__version__"]
-__version__ = "3.6.0"
+__version__ = "3.6.1"
Review Comment:
Th
Taragolis commented on code in PR #37244:
URL: https://github.com/apache/airflow/pull/37244#discussion_r1482832736
##
airflow/providers/airbyte/__init__.py:
##
@@ -27,7 +27,7 @@
__all__ = ["__version__"]
-__version__ = "3.6.0"
+__version__ = "3.6.1"
Review Comment:
You
eladkal commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-1933937121
> The changes in this PR do not use the official Python SDK. This SDK is
still in beta.
Is there official statement of Airbyte saying that? I couldn't find such
statement in the
boring-cyborg[bot] commented on PR #37244:
URL: https://github.com/apache/airflow/pull/37244#issuecomment-1933850459
Congratulations on your first Pull Request and welcome to the Apache Airflow
community! If you have any issues or are unsure about any anything please check
our Contributors'
mitcheccles opened a new pull request, #37244:
URL: https://github.com/apache/airflow/pull/37244
closes: #35011
Updated the Airbyte provider to use the new [Airbyte
API](https://reference.airbyte.com/reference/start). This update ensures
continued functionality as Airbyte
13 matches
Mail list logo
