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

Include the end of the logs when Pulsar trims stdout/stderr. #158

Open
jmchilton opened this issue Feb 14, 2018 · 4 comments
Open

Include the end of the logs when Pulsar trims stdout/stderr. #158

jmchilton opened this issue Feb 14, 2018 · 4 comments

Comments

@jmchilton
Copy link
Member

No description provided.

@jmchilton
Copy link
Member Author

xref #157 (comment)

@natefoo
Copy link
Member

natefoo commented Feb 15, 2018

Can we still plan to send them as job files, or do you feel strongly that these should be part of the message?

@jmchilton
Copy link
Member Author

I think they should be part of the message - I like the API design better that way. I could be wrong though for sure.

@natefoo
Copy link
Member

natefoo commented Feb 15, 2018

In that case, how about we support chunking them? Probably not a bad idea as we want to support streaming while running for user feedback anyway (although sending the full stdout/stderr upon completion is probably safer than having Galaxy use the reassembled pieces).

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

No branches or pull requests

2 participants