[GitHub] [airflow] uranusjr commented on a change in pull request #20011: Type-annotate SkipMixin and BaseXCom

2021-12-05 Thread GitBox
uranusjr commented on a change in pull request #20011: URL: https://github.com/apache/airflow/pull/20011#discussion_r762764645 ## File path: airflow/models/xcom.py ## @@ -99,29 +137,63 @@ def set(cls, key, value, task_id, dag_id, execution_date=None, run_id=None, sess

[GitHub] [airflow] uranusjr closed pull request #20013: Bump sphinx-autoapi to >=1.5

2021-12-05 Thread GitBox
uranusjr closed pull request #20013: URL: https://github.com/apache/airflow/pull/20013 -- 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:

[GitHub] [airflow] uranusjr commented on pull request #20013: Bump sphinx-autoapi to >=1.5

2021-12-05 Thread GitBox
uranusjr commented on pull request #20013: URL: https://github.com/apache/airflow/pull/20013#issuecomment-986516002 Ugh, I give up. Let’s just hide the overloaded definitions from autoapi. (Why are we using autoapi in the first place?) -- This is an automated message from the Apache Git

[GitHub] [airflow] uranusjr commented on a change in pull request #19993: Update upgrading.rst with detailed code example of how to resolve post-upgrade warning

2021-12-05 Thread GitBox
uranusjr commented on a change in pull request #19993: URL: https://github.com/apache/airflow/pull/19993#discussion_r762759725 ## File path: docs/apache-airflow/installation/upgrading.rst ## @@ -79,6 +79,20 @@ table or rename it or move it to another database using those

[GitHub] [airflow] uranusjr commented on pull request #19860: Restore stability and unquarantine all test_scheduler_job tests

2021-12-05 Thread GitBox
uranusjr commented on pull request #19860: URL: https://github.com/apache/airflow/pull/19860#issuecomment-986510211 Why do the query counts change? I don’t see any changes to the main code base. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] eladkal commented on a change in pull request #19993: Update upgrading.rst with detailed code example of how to resolve post-upgrade warning

2021-12-05 Thread GitBox
eladkal commented on a change in pull request #19993: URL: https://github.com/apache/airflow/pull/19993#discussion_r762755536 ## File path: docs/apache-airflow/installation/upgrading.rst ## @@ -79,6 +79,19 @@ table or rename it or move it to another database using those

[GitHub] [airflow] uranusjr commented on issue #19286: metric: Time after triggering

2021-12-05 Thread GitBox
uranusjr commented on issue #19286: URL: https://github.com/apache/airflow/issues/19286#issuecomment-986507656 You need to create a migration file. See https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#metadata-database-updates -- This is an automated message from the Apache

[GitHub] [airflow] EricGao888 edited a comment on issue #19286: metric: Time after triggering

2021-12-05 Thread GitBox
EricGao888 edited a comment on issue #19286: URL: https://github.com/apache/airflow/issues/19286#issuecomment-986380136 > BTW, for backwards compatibility reasons, both the `DagRun` column and `create_dagrun` argument need to be optional (None by default), and if it’s None, we’ll fall

[GitHub] [airflow] uranusjr commented on issue #19967: Breeze: Setup Autocomplete for Breeze

2021-12-05 Thread GitBox
uranusjr commented on issue #19967: URL: https://github.com/apache/airflow/issues/19967#issuecomment-986479329 So when someone adds a new command, they can be automatically reminded to regenerate the autocompletion script. -- This is an automated message from the Apache Git Service. To

[GitHub] [airflow] Bowrna commented on issue #19967: Breeze: Setup Autocomplete for Breeze

2021-12-05 Thread GitBox
Bowrna commented on issue #19967: URL: https://github.com/apache/airflow/issues/19967#issuecomment-986475853 @uranusjr could you explain to me why we have to keep everything in sync using pre-commit? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] uranusjr commented on pull request #20002: Add hour and minute to time format on x-axis of all charts using nvd3.lineChart

2021-12-05 Thread GitBox
uranusjr commented on pull request #20002: URL: https://github.com/apache/airflow/pull/20002#issuecomment-986455946 That looks quite nice to me as well. -- 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

[GitHub] [airflow] uranusjr commented on pull request #19992: Use `python -m pip` to run pip in the new Breeze2

2021-12-05 Thread GitBox
uranusjr commented on pull request #19992: URL: https://github.com/apache/airflow/pull/19992#issuecomment-986455464 > > I think it makes little sense to add the .exe to repo (it's quite big) and everyone should be able to generate one themselves if they need it - with simple one-liner

[GitHub] [airflow] uranusjr commented on issue #19967: Breeze: Setup Autocomplete for Breeze

2021-12-05 Thread GitBox
uranusjr commented on issue #19967: URL: https://github.com/apache/airflow/issues/19967#issuecomment-986442020 A better approach would be to pre-generate completion scripts, and use pre-commit to keep everything in sync. This would eliminate most of the hacks (at runtime at least; all of

[GitHub] [airflow] Bowrna commented on issue #19967: Breeze: Setup Autocomplete for Breeze

2021-12-05 Thread GitBox
Bowrna commented on issue #19967: URL: https://github.com/apache/airflow/issues/19967#issuecomment-986439724 I will pick this issue to work on first @potiuk -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] holly-evans opened a new pull request #20061: Update SlackWebhookHook docstring to recommend password vs. extra

2021-12-05 Thread GitBox
holly-evans opened a new pull request #20061: URL: https://github.com/apache/airflow/pull/20061 ## What Update `SlackWebhookHook` docstring to note that `webhook_token` should go in the `password` field ## Why `webhook_token` in `extra` is deprecated, and you should use

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #20061: Update SlackWebhookHook docstring to recommend password vs. extra

2021-12-05 Thread GitBox
boring-cyborg[bot] commented on pull request #20061: URL: https://github.com/apache/airflow/pull/20061#issuecomment-986436345 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

[GitHub] [airflow] edithturn edited a comment on pull request #19992: Use `python -m pip` to run pip in the new Breeze2

2021-12-05 Thread GitBox
edithturn edited a comment on pull request #19992: URL: https://github.com/apache/airflow/pull/19992#issuecomment-986393525 > I think it makes little sense to add the .exe to repo (it's quite big) and everyone should be able to generate one themselves if they need it - with simple

[GitHub] [airflow] edithturn commented on pull request #19992: Use `python -m pip` to run pip in the new Breeze2

2021-12-05 Thread GitBox
edithturn commented on pull request #19992: URL: https://github.com/apache/airflow/pull/19992#issuecomment-986393525 > I think it makes little sense to add the .exe to repo (it's quite big) and everyone should be able to generate one themselves if they need it - with simple one-liner

[GitHub] [airflow] edithturn commented on pull request #19992: Use `python -m pip` to run pip in the new Breeze2

2021-12-05 Thread GitBox
edithturn commented on pull request #19992: URL: https://github.com/apache/airflow/pull/19992#issuecomment-986390697 I got it @potiuk I tested it again, i didn't give me the same result as you: - git clone https://github.com/potiuk/airflow.git - git checkout

[GitHub] [airflow] josh-fell commented on a change in pull request #19918: Fix mypy errors in example DAGs

2021-12-05 Thread GitBox
josh-fell commented on a change in pull request #19918: URL: https://github.com/apache/airflow/pull/19918#discussion_r762665606 ## File path: airflow/providers/alibaba/cloud/example_dags/example_oss_bucket.py ## @@ -14,6 +14,10 @@ # KIND, either express or implied. See the

[GitHub] [airflow] EricGao888 commented on issue #19286: metric: Time after triggering

2021-12-05 Thread GitBox
EricGao888 commented on issue #19286: URL: https://github.com/apache/airflow/issues/19286#issuecomment-986380136 > BTW, for backwards compatibility reasons, both the `DagRun` column and `create_dagrun` argument need to be optional (None by default), and if it’s None, we’ll fall back to

[GitHub] [airflow] mik-laj commented on issue #20032: Snowflake Provider - Hook's support for not providing a region is broken when using SQLAlchemy

2021-12-05 Thread GitBox
mik-laj commented on issue #20032: URL: https://github.com/apache/airflow/issues/20032#issuecomment-986377533 Feel free to create a PR. I will take care of the review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [airflow] harishkrao commented on issue #20032: Snowflake Provider - Hook's support for not providing a region is broken when using SQLAlchemy

2021-12-05 Thread GitBox
harishkrao commented on issue #20032: URL: https://github.com/apache/airflow/issues/20032#issuecomment-986376341 Thank you for getting back @potiuk. @mik-laj how would you like to proceed? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] potiuk commented on pull request #19860: Restore stability and unquarantine all test_scheduler_job tests

2021-12-05 Thread GitBox
potiuk commented on pull request #19860: URL: https://github.com/apache/airflow/pull/19860#issuecomment-986365402 Looks like this one shoudl be green now ! A lot of flakie's fixed :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[airflow] branch main updated (374574b -> 6e15e3a)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 374574b Fix mypy errors in Microsoft Azure provider (#19923) add 6e15e3a Bug fix in AWS glue operator when

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #19787: Bug fix in AWS glue operator when specifying the WorkerType & NumberOfWorkers

2021-12-05 Thread GitBox
boring-cyborg[bot] commented on pull request #19787: URL: https://github.com/apache/airflow/pull/19787#issuecomment-986363429 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] potiuk closed issue #19207: AWSGlueJobOperator using 'AllocatedCapacity' variable even after specifying the "WorkerType" and "NumberOfWorkers" causing issues with the AWS Glue versi

2021-12-05 Thread GitBox
potiuk closed issue #19207: URL: https://github.com/apache/airflow/issues/19207 -- 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:

[GitHub] [airflow] potiuk merged pull request #19787: Bug fix in AWS glue operator when specifying the WorkerType & NumberOfWorkers

2021-12-05 Thread GitBox
potiuk merged pull request #19787: URL: https://github.com/apache/airflow/pull/19787 -- 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:

[GitHub] [airflow] potiuk commented on pull request #17576: Add pre/post execution hooks

2021-12-05 Thread GitBox
potiuk commented on pull request #17576: URL: https://github.com/apache/airflow/pull/17576#issuecomment-986363233 Another possibly interesting use case for pre_execute: https://apache-airflow.slack.com/archives/CCVDRN0F9/p1638740065139100 Basically adding a "session"

[GitHub] [airflow] eladkal commented on pull request #20044: Add ShortCircuitOperator configurability for respecting downstream trigger rules

2021-12-05 Thread GitBox
eladkal commented on pull request #20044: URL: https://github.com/apache/airflow/pull/20044#issuecomment-986351985 @kazanzhy read what I wrote in https://github.com/apache/airflow/issues/7858 -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] kazanzhy commented on pull request #20044: Add ShortCircuitOperator configurability for respecting downstream trigger rules

2021-12-05 Thread GitBox
kazanzhy commented on pull request #20044: URL: https://github.com/apache/airflow/pull/20044#issuecomment-986347439 Very confusing feature, I think. I just trying to figure out how it will work. Let's imagine the next DAG: ``` shortcircuit >> [task_1, task_2] [task_1,

[GitHub] [airflow] hwooson12 commented on pull request #20002: Add hour and minute to time format on x-axis of all charts using nvd3.lineChart

2021-12-05 Thread GitBox
hwooson12 commented on pull request #20002: URL: https://github.com/apache/airflow/pull/20002#issuecomment-986343911 I realized it's not that simple to break the x-axis labels into multiple lines. It might need to fix the external library like nvd3. The only thing I can modify to change

[airflow] branch main updated (564fe63 -> 374574b)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 564fe63 Added example DAG for MSSQL to Google Cloud Storage (GCS) (#19873) add 374574b Fix mypy errors in

[airflow] branch main updated (8ca3eda -> 564fe63)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 8ca3eda Fixing MyPy issues in testa/jobs (#19998) add 564fe63 Added example DAG for MSSQL to Google Cloud

[GitHub] [airflow] potiuk merged pull request #19923: Fix mypy errors in Microsoft Azure provider

2021-12-05 Thread GitBox
potiuk merged pull request #19923: URL: https://github.com/apache/airflow/pull/19923 -- 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:

[GitHub] [airflow] potiuk merged pull request #19873: Added example DAG for MSSQL to Google Cloud Storage (GCS)

2021-12-05 Thread GitBox
potiuk merged pull request #19873: URL: https://github.com/apache/airflow/pull/19873 -- 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:

[GitHub] [airflow] potiuk commented on a change in pull request #19918: Fix mypy errors in example DAGs

2021-12-05 Thread GitBox
potiuk commented on a change in pull request #19918: URL: https://github.com/apache/airflow/pull/19918#discussion_r762640851 ## File path: airflow/providers/alibaba/cloud/example_dags/example_oss_bucket.py ## @@ -14,6 +14,10 @@ # KIND, either express or implied. See the

[GitHub] [airflow] potiuk commented on pull request #19953: update upper bound for MarkupSafe

2021-12-05 Thread GitBox
potiuk commented on pull request #19953: URL: https://github.com/apache/airflow/pull/19953#issuecomment-986336513 You can easily reproduce it in Breeze - (see the error log for instructions - you can also build your own image with `breeze --upgrade-to-newer-dependencies` in your PR and

[GitHub] [airflow] potiuk commented on pull request #19953: update upper bound for MarkupSafe

2021-12-05 Thread GitBox
potiuk commented on pull request #19953: URL: https://github.com/apache/airflow/pull/19953#issuecomment-986336013 Looks like ``` tests/www/views/test_views_rendered.py::test_user_defined_filter_and_macros_raise_error: AssertionError ``` Is the one that should be

[GitHub] [airflow] potiuk commented on issue #19957: AIrflow crashes with a psycopg2.errors.DeadlockDetected exception

2021-12-05 Thread GitBox
potiuk commented on issue #19957: URL: https://github.com/apache/airflow/issues/19957#issuecomment-986335228 (Reason why I am asking - there are very low number of Postgres deadlock reports we receive, because (unlike MySQL) Postgres usually shows "real" deadlocks and we'd expect this

[GitHub] [airflow] potiuk commented on issue #19957: AIrflow crashes with a psycopg2.errors.DeadlockDetected exception

2021-12-05 Thread GitBox
potiuk commented on issue #19957: URL: https://github.com/apache/airflow/issues/19957#issuecomment-986334766 Also - is there a chance to find out from the logs what was the "other" query that the deadlock happened on? Is this also possible that you have another person or process (either

[GitHub] [airflow] potiuk commented on issue #19957: AIrflow crashes with a psycopg2.errors.DeadlockDetected exception

2021-12-05 Thread GitBox
potiuk commented on issue #19957: URL: https://github.com/apache/airflow/issues/19957#issuecomment-986332738 Do you happen to have one or multiple schedulers? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] potiuk commented on pull request #19963: Add possibility to create user in the Remote User mode Auth.

2021-12-05 Thread GitBox
potiuk commented on pull request #19963: URL: https://github.com/apache/airflow/pull/19963#issuecomment-986330459 I think some common approach here should be worked out. I am for adding it, but I am not as "deeply" in the rules and configuration - I believe for the new UI we are porting

[GitHub] [airflow] github-actions[bot] closed issue #18114: TaskFlow API DAGs not working when the files are inside a .zip file.

2021-12-05 Thread GitBox
github-actions[bot] closed issue #18114: URL: https://github.com/apache/airflow/issues/18114 -- 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:

[GitHub] [airflow] github-actions[bot] commented on issue #18114: TaskFlow API DAGs not working when the files are inside a .zip file.

2021-12-05 Thread GitBox
github-actions[bot] commented on issue #18114: URL: https://github.com/apache/airflow/issues/18114#issuecomment-986329841 This issue has been closed because it has not received response from the issue author. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] github-actions[bot] commented on issue #19136: Variable not found, causing broken DAGs, and cryptography error

2021-12-05 Thread GitBox
github-actions[bot] commented on issue #19136: URL: https://github.com/apache/airflow/issues/19136#issuecomment-986329813 This issue has been automatically marked as stale because it has been open for 30 days with no response from the author. It will be closed in next 7 days if no further

[GitHub] [airflow] github-actions[bot] commented on pull request #19107: Added notion hook

2021-12-05 Thread GitBox
github-actions[bot] commented on pull request #19107: URL: https://github.com/apache/airflow/pull/19107#issuecomment-986329819 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

[GitHub] [airflow] github-actions[bot] commented on pull request #18949: Fix very rare test_fast_follow failure.

2021-12-05 Thread GitBox
github-actions[bot] commented on pull request #18949: URL: https://github.com/apache/airflow/pull/18949#issuecomment-986329835 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

[GitHub] [airflow] potiuk commented on pull request #19689: add redis runAsUser option (#19681)

2021-12-05 Thread GitBox
potiuk commented on pull request #19689: URL: https://github.com/apache/airflow/pull/19689#issuecomment-986327109 Sorry - needs rebase again (but we had some intermittent problems that should nearly be gone - so please - rebase). -- This is an automated message from the Apache Git

[GitHub] [airflow] potiuk commented on issue #19983: Exclude airflow runner internals from Operator failure tracebacks

2021-12-05 Thread GitBox
potiuk commented on issue #19983: URL: https://github.com/apache/airflow/issues/19983#issuecomment-986326284 I think it depends on whose view you take into account, I am for adding as much information as possible in the stack trace, because you never know when more stack trace is useful.

[GitHub] [airflow] potiuk commented on issue #19995: TaskInstance.log_url is uncallable due to execution_date being deprecated

2021-12-05 Thread GitBox
potiuk commented on issue #19995: URL: https://github.com/apache/airflow/issues/19995#issuecomment-986323373 TaskInstance is just a reflection of datamodel, so it is bound to change. See the latest definition in Airflow code (I gues you initially looked at internals too to find out how to

[GitHub] [airflow] potiuk closed issue #19995: TaskInstance.log_url is uncallable due to execution_date being deprecated

2021-12-05 Thread GitBox
potiuk closed issue #19995: URL: https://github.com/apache/airflow/issues/19995 -- 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:

[airflow] branch main updated: Fixing MyPy issues in testa/jobs (#19998)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 8ca3eda Fixing MyPy issues in testa/jobs (#19998)

[GitHub] [airflow] potiuk merged pull request #19998: Fixing MyPy issues in tests/jobs

2021-12-05 Thread GitBox
potiuk merged pull request #19998: URL: https://github.com/apache/airflow/pull/19998 -- 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:

[GitHub] [airflow] potiuk commented on pull request #19998: Fixing MyPy issues in tests/jobs

2021-12-05 Thread GitBox
potiuk commented on pull request #19998: URL: https://github.com/apache/airflow/pull/19998#issuecomment-986321791 Intermittent errors only. Merging. -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #19998: Fixing MyPy issues in tests/jobs

2021-12-05 Thread GitBox
github-actions[bot] commented on pull request #19998: URL: https://github.com/apache/airflow/pull/19998#issuecomment-986321738 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and

[GitHub] [airflow] potiuk commented on pull request #20001: Fix BigQuery to MSSQL transfer issue

2021-12-05 Thread GitBox
potiuk commented on pull request #20001: URL: https://github.com/apache/airflow/pull/20001#issuecomment-986321440 The tests are failing and likely related to your chanves - examples are not importable -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] potiuk commented on issue #12726: Add classic operator in TaskFlow API tutorial

2021-12-05 Thread GitBox
potiuk commented on issue #12726: URL: https://github.com/apache/airflow/issues/12726#issuecomment-986320786 Fine for me. -- 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.

[GitHub] [airflow] potiuk closed issue #12726: Add classic operator in TaskFlow API tutorial

2021-12-05 Thread GitBox
potiuk closed issue #12726: URL: https://github.com/apache/airflow/issues/12726 -- 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:

[GitHub] [airflow] potiuk commented on issue #20004: Airflow UI "stuck" and not showing most recent change

2021-12-05 Thread GitBox
potiuk commented on issue #20004: URL: https://github.com/apache/airflow/issues/20004#issuecomment-986320371 I think you need to dig deeper. The docker-compose is just a quick start and you are on your own to make any modifications. You have your own modifications in tehre, local dags,

[GitHub] [airflow] potiuk commented on issue #20007: PostgresToGCSOperator fail on empty table and use_server_side_cursor=True

2021-12-05 Thread GitBox
potiuk commented on issue #20007: URL: https://github.com/apache/airflow/issues/20007#issuecomment-986318107 Maybe you would like to contribute a PR to fix it @PikaYellow35 - we have > 1800 contributors, you can become one of them :) -- This is an automated message from the Apache Git

[airflow] branch main updated: Remove postgres 9.6 support (#19987)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new a299cbf Remove postgres 9.6 support (#19987)

[GitHub] [airflow] potiuk closed issue #19986: Postgres 9.6 end of support

2021-12-05 Thread GitBox
potiuk closed issue #19986: URL: https://github.com/apache/airflow/issues/19986 -- 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:

[GitHub] [airflow] potiuk merged pull request #19987: Remove postgres 9.6 support

2021-12-05 Thread GitBox
potiuk merged pull request #19987: URL: https://github.com/apache/airflow/pull/19987 -- 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:

[airflow] branch main updated: Add xcom clearing behaviour on task retries (#19968)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 538612c Add xcom clearing behaviour on task

[GitHub] [airflow] potiuk closed issue #18917: Document xcom clearing behaviour on task retries

2021-12-05 Thread GitBox
potiuk closed issue #18917: URL: https://github.com/apache/airflow/issues/18917 -- 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:

[GitHub] [airflow] potiuk merged pull request #19968: Add xcom clearing behaviour on task retries

2021-12-05 Thread GitBox
potiuk merged pull request #19968: URL: https://github.com/apache/airflow/pull/19968 -- 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:

[GitHub] [airflow] potiuk commented on a change in pull request #20011: Type-annotate SkipMixin and BaseXCom

2021-12-05 Thread GitBox
potiuk commented on a change in pull request #20011: URL: https://github.com/apache/airflow/pull/20011#discussion_r762628786 ## File path: airflow/models/xcom.py ## @@ -376,4 +505,7 @@ def resolve_xcom_backend(): return BaseXCom -XCom = resolve_xcom_backend() +if

[GitHub] [airflow] potiuk commented on pull request #19990: Order bug report fields by importance

2021-12-05 Thread GitBox
potiuk commented on pull request #19990: URL: https://github.com/apache/airflow/pull/19990#issuecomment-986313600 Good for me with Airflow version too. -- 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

[airflow] branch main updated: Fixing mypy issues inside tests model (#20026)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new e8b1120 Fixing mypy issues inside tests model

[GitHub] [airflow] potiuk merged pull request #20026: Fixing mypy issues inside tests model

2021-12-05 Thread GitBox
potiuk merged pull request #20026: URL: https://github.com/apache/airflow/pull/20026 -- 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:

[airflow] branch main updated: Inrease length of the email and username (#19932)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new 7068341 Inrease length of the email and username

[GitHub] [airflow] potiuk commented on pull request #19932: Increase length of the email and username

2021-12-05 Thread GitBox
potiuk commented on pull request #19932: URL: https://github.com/apache/airflow/pull/19932#issuecomment-986312717 Only internittent errors that are being fixed already. Merging -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] potiuk merged pull request #19932: Increase length of the email and username

2021-12-05 Thread GitBox
potiuk merged pull request #19932: URL: https://github.com/apache/airflow/pull/19932 -- 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:

[GitHub] [airflow] potiuk commented on issue #20033: Transfer operators for azure blob storage

2021-12-05 Thread GitBox
potiuk commented on issue #20033: URL: https://github.com/apache/airflow/issues/20033#issuecomment-986312280 Feel free! -- 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

[GitHub] [airflow] potiuk commented on issue #19891: Re-enable MyPy

2021-12-05 Thread GitBox
potiuk commented on issue #19891: URL: https://github.com/apache/airflow/issues/19891#issuecomment-986312131 > Anyone grab `airflow/providers/salesforce` yet? Feel free! . few more merges and I am going to refresh the list to see where we are, on top of manual updatees. -- This

[jira] [Commented] (AIRFLOW-4922) If a task crashes, host name is not committed to the database so logs aren't able to be seen in the UI

2021-12-05 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-4922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17453706#comment-17453706 ] ASF GitHub Bot commented on AIRFLOW-4922: - potiuk edited a comment on pull request #6722: URL:

[jira] [Commented] (AIRFLOW-4922) If a task crashes, host name is not committed to the database so logs aren't able to be seen in the UI

2021-12-05 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-4922?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17453705#comment-17453705 ] ASF GitHub Bot commented on AIRFLOW-4922: - potiuk commented on pull request #6722: URL:

[GitHub] [airflow] potiuk edited a comment on pull request #6722: [AIRFLOW-4922]Fix task get log by Web UI

2021-12-05 Thread GitBox
potiuk edited a comment on pull request #6722: URL: https://github.com/apache/airflow/pull/6722#issuecomment-986311760 Well. @blotouta2. I think this issue discusses 3 or 4 different problems. So your comment is pretty meaningless. Also the issue is clsoed so it's likely it's a different

[GitHub] [airflow] potiuk commented on pull request #6722: [AIRFLOW-4922]Fix task get log by Web UI

2021-12-05 Thread GitBox
potiuk commented on pull request #6722: URL: https://github.com/apache/airflow/pull/6722#issuecomment-986311760 Well. @blotouta2. I think this issue discusses 3 or 4 different. Problems. The issue is clsoed so it's likely it's a different issue altogether. If you REALLLY want to get

[GitHub] [airflow] potiuk commented on issue #18599: datadog parsing error for dagrun.schedule_delay since it is not passed in float type

2021-12-05 Thread GitBox
potiuk commented on issue #18599: URL: https://github.com/apache/airflow/issues/18599#issuecomment-986310634 Which PR? -- 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

[GitHub] [airflow] potiuk commented on pull request #20058: Draft: Add Qubole how to documentation

2021-12-05 Thread GitBox
potiuk commented on pull request #20058: URL: https://github.com/apache/airflow/pull/20058#issuecomment-986310341 Nice @chattarajoy waiting for your review to merge :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [airflow] kazanzhy commented on issue #10929: Add how to documentation for Qubole Operator

2021-12-05 Thread GitBox
kazanzhy commented on issue #10929: URL: https://github.com/apache/airflow/issues/10929#issuecomment-986310062 Hi all. I've prepared a draft PR #20058 I will be very glad to any suggestions, especially from @chattarajoy -- This is an automated message from the Apache Git

[GitHub] [airflow] kazanzhy opened a new pull request #20058: Draft: Add Qubole how to documentation

2021-12-05 Thread GitBox
kazanzhy opened a new pull request #20058: URL: https://github.com/apache/airflow/pull/20058 Add draft of Qubole how-to documentation for Qubole Operators and Sensors I will be very glad to any suggestions, especially from Qubole users. closes: #10929 -- This is an automated

[GitHub] [airflow] potiuk merged pull request #19985: Add show dag dependencies feature to CLI

2021-12-05 Thread GitBox
potiuk merged pull request #19985: URL: https://github.com/apache/airflow/pull/19985 -- 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:

[GitHub] [airflow] potiuk closed issue #19080: Display DAGs dependencies in CLI

2021-12-05 Thread GitBox
potiuk closed issue #19080: URL: https://github.com/apache/airflow/issues/19080 -- 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:

[airflow] branch main updated (728e94a -> 498f98a)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 728e94a Refactor DatabricksHook (#19835) add 498f98a Add show dag dependencies feature to CLI (#19985) No new

[GitHub] [airflow] potiuk commented on pull request #19985: Add show dag dependencies feature to CLI

2021-12-05 Thread GitBox
potiuk commented on pull request #19985: URL: https://github.com/apache/airflow/pull/19985#issuecomment-986309521 Intermittent task failure likely already handled in #19860 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] potiuk commented on issue #20032: Snowflake Provider - Hook's support for not providing a region is broken when using SQLAlchemy

2021-12-05 Thread GitBox
potiuk commented on issue #20032: URL: https://github.com/apache/airflow/issues/20032#issuecomment-986308802 But maybe @mik-laj would like to do it firsst - so agree between yourselves ;) -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] potiuk commented on issue #20032: Snowflake Provider - Hook's support for not providing a region is broken when using SQLAlchemy

2021-12-05 Thread GitBox
potiuk commented on issue #20032: URL: https://github.com/apache/airflow/issues/20032#issuecomment-986308709 Sure @harishkrao . Assigned you. -- 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

[GitHub] [airflow] github-actions[bot] commented on pull request #19985: Add show dag dependencies feature to CLI

2021-12-05 Thread GitBox
github-actions[bot] commented on pull request #19985: URL: https://github.com/apache/airflow/pull/19985#issuecomment-986308053 The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and

[GitHub] [airflow] potiuk commented on pull request #19985: Add show dag dependencies feature to CLI

2021-12-05 Thread GitBox
potiuk commented on pull request #19985: URL: https://github.com/apache/airflow/pull/19985#issuecomment-986307979 Nice! -- 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

[airflow] branch main updated (af28b41 -> 728e94a)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from af28b41 Add sensor for AWS Batch (#19850) (#19885) add 728e94a Refactor DatabricksHook (#19835) No new

[GitHub] [airflow] potiuk merged pull request #19835: Refactor DatabricksHook

2021-12-05 Thread GitBox
potiuk merged pull request #19835: URL: https://github.com/apache/airflow/pull/19835 -- 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:

[GitHub] [airflow] potiuk commented on pull request #19156: Databricks operators: added support for pipeline task, jar_params, example DAG for RunNowOperator, and updated docs

2021-12-05 Thread GitBox
potiuk commented on pull request #19156: URL: https://github.com/apache/airflow/pull/19156#issuecomment-986307003 Needs rebase! -- 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

[airflow] branch main updated: Add sensor for AWS Batch (#19850) (#19885)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/main by this push: new af28b41 Add sensor for AWS Batch (#19850)

[GitHub] [airflow] potiuk merged pull request #19885: Add sensor for AWS Batch (#19850)

2021-12-05 Thread GitBox
potiuk merged pull request #19885: URL: https://github.com/apache/airflow/pull/19885 -- 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:

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #19885: Add sensor for AWS Batch (#19850)

2021-12-05 Thread GitBox
boring-cyborg[bot] commented on pull request #19885: URL: https://github.com/apache/airflow/pull/19885#issuecomment-986306924 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] potiuk closed issue #19850: AWS Batch Job Sensor

2021-12-05 Thread GitBox
potiuk closed issue #19850: URL: https://github.com/apache/airflow/issues/19850 -- 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:

[airflow] branch main updated (9319a31 -> 7627de3)

2021-12-05 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/airflow.git. from 9319a31 Fixing failing quarantined test cases in test_task_command (#19864) add 7627de3 Add requirements.txt

  1   2   >