Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug - Invalid type for variable for execution_timeout and dag_run_timeout #53

Merged
merged 3 commits into from
Dec 4, 2022

Conversation

DavidKatz-il
Copy link
Contributor

@DavidKatz-il DavidKatz-il commented Nov 27, 2022

Resolves #52

@DavidKatz-il DavidKatz-il changed the title Fix-bug-invalid-type-variable Fix bug - Invalid type for variable for execution_timeout and dag_run_timeout Nov 27, 2022
@potiuk
Copy link
Member

potiuk commented Dec 4, 2022

Can you please rebase @DavidKatz-il ? There were out-dated Python 3.6 version that got removed and I just added proper versions in the CI.

As Python version 3.6 is removed, we should also update Python
versions used for building the Python Client in CI.
@DavidKatz-il
Copy link
Contributor Author

Can you please rebase @DavidKatz-il ? There were out-dated Python 3.6 version that got removed and I just added proper versions in the CI.

Done

@potiuk potiuk merged commit f32517a into apache:master Dec 4, 2022
@DavidKatz-il DavidKatz-il deleted the fix-bug-invalid-type-variable branch December 4, 2022 19:10
ephraimbuddy pushed a commit that referenced this pull request May 4, 2023
ephraimbuddy added a commit that referenced this pull request May 4, 2023
…_timeout (#53) (#76)

(cherry picked from commit f32517a)

Co-authored-by: David Katz <[email protected]>
ephraimbuddy added a commit that referenced this pull request May 17, 2023
…_timeout (#53) (#76)

(cherry picked from commit f32517a)

Co-authored-by: David Katz <[email protected]>
(cherry picked from commit ddd6fc0)
ephraimbuddy added a commit that referenced this pull request May 17, 2023
* Add Client Version 2.6.1

* Fix bug - Invalid type for variable for execution_timeout and dag_run_timeout (#53) (#76)

(cherry picked from commit f32517a)

Co-authored-by: David Katz <[email protected]>
(cherry picked from commit ddd6fc0)
pierrejeambrun pushed a commit that referenced this pull request Jun 21, 2023
…_timeout (#53) (#76)

(cherry picked from commit f32517a)

Co-authored-by: David Katz <[email protected]>
(cherry picked from commit ddd6fc0)
pierrejeambrun added a commit that referenced this pull request Jun 21, 2023
…_timeout (#53) (#76) (#87)

(cherry picked from commit f32517a)

Co-authored-by: David Katz <[email protected]>
(cherry picked from commit ddd6fc0)

Co-authored-by: Ephraim Anierobi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: DAGApi.get_dag_details fails if DAG Run Timeout is None
2 participants