alizademhdi closed pull request #38628: Fix inital value for _batch_id
attribute in livy operator
URL: https://github.com/apache/airflow/pull/38628
--
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 th
github-actions[bot] commented on PR #38628:
URL: https://github.com/apache/airflow/pull/38628#issuecomment-2131753717
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
mateuslatrova commented on PR #38628:
URL: https://github.com/apache/airflow/pull/38628#issuecomment-2048507160
Thanks for the idea, @potiuk! That makes much more sense. I will try to
implement it.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
potiuk commented on PR #38628:
URL: https://github.com/apache/airflow/pull/38628#issuecomment-2036432683
Trigger should (even if it times out) produce a trigger event with batch_id
information in it. So Airflow should not kill trigger, it should timeout itself
before Airflow does it and sig
mateuslatrova commented on PR #38628:
URL: https://github.com/apache/airflow/pull/38628#issuecomment-2035812370
Hi! I was also working on issue #37898 , and one of the requirements the
author mentioned was:
> "The Livy operator should fail due to a timeout, and simultaneously, it
sh
alizademhdi commented on PR #38628:
URL: https://github.com/apache/airflow/pull/38628#issuecomment-2030790977
Sure. I will try.
--
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 commen
eladkal commented on PR #38628:
URL: https://github.com/apache/airflow/pull/38628#issuecomment-2028687443
Can you add unit test to avoid regression?
--
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 t
alizademhdi commented on PR #38628:
URL: https://github.com/apache/airflow/pull/38628#issuecomment-2028073596
Fixed static tests :D
--
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 c
alizademhdi opened a new pull request, #38628:
URL: https://github.com/apache/airflow/pull/38628
### Related issue
#37898
### Explain
when the livy operator is rewaked up after trigger is time out, the
_batch_id is not initialized, which cause it to raise this exeption:
`