boring-cyborg[bot] commented on PR #37458:
URL: https://github.com/apache/airflow/pull/37458#issuecomment-2010010788
Awesome work, congrats on your first merged pull request! You are invited to
check our [Issue Tracker](https://github.com/apache/airflow/issues) for
additional
Taragolis merged PR #37458:
URL: https://github.com/apache/airflow/pull/37458
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail:
uzhastik commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1528362006
##
airflow/providers/yandex/provider.yaml:
##
Review Comment:
@potiuk I'm working in minimal context - single provider. And I have problem
with describing
potiuk commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1525095700
##
airflow/providers/yandex/provider.yaml:
##
Review Comment:
But maybe that's what you indeed had in mind -> this is the description of
what we plan to do
potiuk commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1525091341
##
airflow/providers/yandex/provider.yaml:
##
Review Comment:
Contributions are welcome. What exactly test did you have in mind to detect
it @uzhastik ? How
potiuk commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1525091341
##
airflow/providers/yandex/provider.yaml:
##
Review Comment:
Contributions are welcome. What exactly test did you have in mind to detect
it @uzhastik ? How
uzhastik commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1525050181
##
airflow/providers/yandex/provider.yaml:
##
Review Comment:
it would be nice to have some tests to detect problems in dependencies
configuration. for
uzhastik commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1524882892
##
airflow/providers/yandex/operators/yandexcloud_yq.py:
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
uzhastik commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1524713996
##
airflow/providers/yandex/operators/yandexcloud_yq.py:
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Taragolis commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1524627620
##
airflow/providers/yandex/operators/yandexcloud_yq.py:
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Taragolis commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1524619767
##
airflow/providers/yandex/operators/yandexcloud_yq.py:
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Taragolis commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1524615923
##
airflow/providers/yandex/operators/yandexcloud_yq.py:
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
uzhastik commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1523224271
##
airflow/providers/yandex/operators/yandexcloud_yq.py:
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
uzhastik commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1523224271
##
airflow/providers/yandex/operators/yandexcloud_yq.py:
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
Taragolis commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1522956524
##
airflow/providers/yandex/operators/yandexcloud_yq.py:
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
uzhastik commented on PR #37458:
URL: https://github.com/apache/airflow/pull/37458#issuecomment-1993994426
@Taragolis @potiuk @eladkal @rozza @ashb @jedcunningham ping again =)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
uzhastik commented on PR #37458:
URL: https://github.com/apache/airflow/pull/37458#issuecomment-1988008811
@Taragolis @potiuk @eladkal @rozza @ashb @jedcunningham hello, team! finally
PR is green. please check it out! =))
--
This is an automated message from the Apache Git Service.
To
uzhastik commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1509972280
##
airflow/providers/yandex/hooks/http_client.py:
##
@@ -0,0 +1,321 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license
potiuk commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1498408740
##
airflow/providers/yandex/hooks/http_client.py:
##
@@ -0,0 +1,321 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license
uzhastik commented on PR #37458:
URL: https://github.com/apache/airflow/pull/37458#issuecomment-1948914923
> I'm not familiar with Yandex cloud and their API, but is any chance that
Yandex Query supported by [`yandexcloud`](https://pypi.org/project/yandexcloud/)
>
> cc: @Piatachock
uzhastik commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1492377851
##
airflow/providers/yandex/hooks/http_client.py:
##
@@ -0,0 +1,321 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license
Taragolis commented on PR #37458:
URL: https://github.com/apache/airflow/pull/37458#issuecomment-1947076850
And static checks and mypy should be fixed:
-
https://github.com/apache/airflow/actions/runs/7919541474/job/21623012828?pr=37458
-
Taragolis commented on code in PR #37458:
URL: https://github.com/apache/airflow/pull/37458#discussion_r1491369765
##
airflow/providers/yandex/provider.yaml:
##
@@ -62,25 +61,41 @@ integrations:
logo: /integration-logos/yandex/Yandex-Cloud.png
tags: [service]
+ -
boring-cyborg[bot] commented on PR #37458:
URL: https://github.com/apache/airflow/pull/37458#issuecomment-1946570818
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
uzhastik opened a new pull request, #37458:
URL: https://github.com/apache/airflow/pull/37458
---
Add support for Yandex Query service in Yandex Cloud. Ability to run
federative queries in Yandex Cloud.
--
This is an automated message from the Apache Git
25 matches
Mail list logo