Skip to content

Conversation

@yongjiaw
Copy link
Contributor

No description provided.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@JoshRosen
Copy link
Contributor

Hey @yongjiaw, thanks for submitting this! I've been wanting to do something like this myself, so I plan to take a look at this PR and will try it out.

@yongjiaw
Copy link
Contributor Author

@JoshRosen, I started this as a hack. The only thing I don't feel good about it is I have to remove the logType checks from LogPage.scala, which runs with the worker process and is not aware of the executor's loggers. It works for the driver's executorPage because the driver is already listening for executor data. Similarly, the worker page right now only has hard-coded links to stdout and stderr. Maybe this can be further improved by letting worker listen to executors. This small step should be useful enough.

@yongjiaw
Copy link
Contributor Author

this PR had some issue, I created another one. Closing this one.
#9321

@yongjiaw yongjiaw closed this Oct 28, 2015
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

Successfully merging this pull request may close these issues.

3 participants