uranusjr commented on code in PR #33898:
URL: https://github.com/apache/airflow/pull/33898#discussion_r1310006395
##########
airflow/providers/apache/spark/hooks/spark_submit.py:
##########
@@ -461,31 +461,31 @@ def _process_spark_submit_log(self, itr: Iterator[Any])
-> None:
# If we run yarn cluster mode, we want to extract the application
id from
# the logs so we can kill the application when we stop it
unexpectedly
if self._is_yarn and self._connection["deploy_mode"] == "cluster":
- match = re.search("(application[0-9_]+)", line)
+ match = re.search("application[0-9_]+", line)
if match:
- self._yarn_application_id = match.groups()[0]
+ self._yarn_application_id = match.group(0)
Review Comment:
Should this be `group(1)` instead?
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
