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

Retrieve session logs when using Livy Operator #17340

Closed
collinmcnulty opened this issue Jul 30, 2021 · 4 comments · Fixed by #17393
Closed

Retrieve session logs when using Livy Operator #17340

collinmcnulty opened this issue Jul 30, 2021 · 4 comments · Fixed by #17393
Assignees
Labels

Comments

@collinmcnulty
Copy link
Contributor

Description
The Airflow logs generated by the Livy operator currently only state the status of the submitted batch. To view the logs from the job itself, one must go separately to the session logs. I think that Airflow should have the option (possibly on by default) that retrieves the session logs after the batch reaches a terminal state if a polling_interval has been set.

Use case / motivation
When debugging a task submitted via Livy, the session logs are the first place to check. For most other tasks, including SparkSubmitOperator, viewing the first-check logs can be done in the Airflow UI, but for Livy you must go externally or write a separate task to retrieve them.

Are you willing to submit a PR?

I don't yet have a good sense of how challenging this will be to set up and test. I can try but if anyone else wants to go for it, don't let my attempt stop you.

Related Issues

None I could find

@collinmcnulty collinmcnulty added the kind:feature Feature Requests label Jul 30, 2021
@boring-cyborg
Copy link

boring-cyborg bot commented Jul 30, 2021

Thanks for opening your first issue here! Be sure to follow the issue template!

@sreenath-kamath
Copy link
Contributor

I can submit a PR for this, Will be a good addition to the LivyOpeartor

@potiuk
Copy link
Member

potiuk commented Jul 30, 2021

Assigned you @sreenath-kamath !

@sreenath-kamath
Copy link
Contributor

Have raised a PR #17393 for the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants