Skip to content

Commit

Permalink
logger.info instead of print statement in my airflow bootstrapper cod…
Browse files Browse the repository at this point in the history
…e as well

Signed-off-by: shalberd <21118431+shalberd@users.noreply.github.com>
  • Loading branch information
shalberd committed Aug 19, 2024
1 parent 6bcc7ee commit fe45107
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions elyra/airflow/bootstrapper.py
Original file line number Diff line number Diff line change
Expand Up @@ -353,8 +353,8 @@ def execute(self) -> None:
if enable_generic_node_script_output_to_s3:
with open(python_script_output, "w") as log_file:
log_file.write(output)
logger.info(f"Output: {output}")
logger.info(f"Return code: {result.returncode}")
logger.info("Output: %s", output)
logger.info("Return code: %s", result.returncode)
except subprocess.CalledProcessError as e:
logger.error("Output: %s", e.output.decode("utf-8"))
logger.error("Return code: %s", e.returncode)
Expand Down Expand Up @@ -398,8 +398,8 @@ def execute(self) -> None:
if enable_generic_node_script_output_to_s3:
with open(r_script_output, "w") as log_file:
log_file.write(output)
logger.info(f"Output: {output}")
logger.info(f"Return code: {result.returncode}")
logger.info("Output: %s", output)
logger.info("Return code: %s", result.returncode)
except subprocess.CalledProcessError as e:
logger.error("Output: %s", e.output.decode("utf-8"))
logger.error("Return code: %s", e.returncode)
Expand Down

0 comments on commit fe45107

Please sign in to comment.