Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-24 Thread via GitHub
zeotuan commented on PR #34386: URL: https://github.com/apache/airflow/pull/34386#issuecomment-1778569490 Oh Yes we do. I will submit a PR for that -- 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

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-23 Thread via GitHub
eladkal commented on PR #34386: URL: https://github.com/apache/airflow/pull/34386#issuecomment-1776612174 @zeotuan do we need to expose this parameter in SparkSubmitOperator as well? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-21 Thread via GitHub
eladkal merged PR #34386: URL: https://github.com/apache/airflow/pull/34386 -- 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: commits-unsubscr...@airflow.

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-19 Thread via GitHub
zeotuan commented on code in PR #34386: URL: https://github.com/apache/airflow/pull/34386#discussion_r1366370513 ## airflow/security/kerberos.py: ## @@ -59,10 +63,7 @@ def renew_from_kt(principal: str | None, keytab: str, exit_on_fail: bool = True) # minutes to give oursel

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-18 Thread via GitHub
eladkal commented on code in PR #34386: URL: https://github.com/apache/airflow/pull/34386#discussion_r1364524089 ## airflow/security/kerberos.py: ## @@ -59,10 +63,7 @@ def renew_from_kt(principal: str | None, keytab: str, exit_on_fail: bool = True) # minutes to give oursel

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-13 Thread via GitHub
zeotuan commented on code in PR #34386: URL: https://github.com/apache/airflow/pull/34386#discussion_r1359200998 ## airflow/security/kerberos.py: ## @@ -59,10 +63,7 @@ def renew_from_kt(principal: str | None, keytab: str, exit_on_fail: bool = True) # minutes to give oursel

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-13 Thread via GitHub
eladkal commented on code in PR #34386: URL: https://github.com/apache/airflow/pull/34386#discussion_r1358077855 ## airflow/security/kerberos.py: ## @@ -59,10 +63,7 @@ def renew_from_kt(principal: str | None, keytab: str, exit_on_fail: bool = True) # minutes to give oursel

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-05 Thread via GitHub
zeotuan commented on code in PR #34386: URL: https://github.com/apache/airflow/pull/34386#discussion_r1348191633 ## airflow/providers/apache/spark/hooks/spark_submit.py: ## @@ -111,6 +111,7 @@ def __init__( executor_memory: str | None = None, driver_memory: str

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-03 Thread via GitHub
Lee-W commented on code in PR #34386: URL: https://github.com/apache/airflow/pull/34386#discussion_r1345154160 ## airflow/providers/apache/spark/hooks/spark_submit.py: ## @@ -111,6 +111,7 @@ def __init__( executor_memory: str | None = None, driver_memory: str |

Re: [PR] Add use_krb5ccache option to SparkSubmitHook [airflow]

2023-10-03 Thread via GitHub
Lee-W commented on PR #34386: URL: https://github.com/apache/airflow/pull/34386#issuecomment-1746086068 I believe the CI failure in this branch has been resolved by https://github.com/apache/airflow/pull/34701. Could you please rebase the branch and see whether it passes the CI? Thanks! -