Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-20 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-20 Thread via GitHub
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:

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-18 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-14 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-13 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-13 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-11 Thread via 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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-03-02 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-02-21 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-02-16 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-02-16 Thread via GitHub
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

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-02-15 Thread via GitHub
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 -

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-02-15 Thread via GitHub
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] + -

Re: [PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-02-15 Thread via GitHub
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

[PR] Add Yandex Query support from Yandex.Cloud [airflow]

2024-02-15 Thread via GitHub
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