Github user CodingCat commented on the pull request:
https://github.com/apache/spark/pull/7910#issuecomment-150359302
@JoshRosen @squito
I adjusted the patch according to Josh's suggestions,
now the search is covering all columns (I thought it only cares about
stacktrace column)
the stacktrace is shown as a new row
For the StackTrace row, the current implementation is to <b>add a new <tr>
following the row showing Thread ID, Name and State, instead of toggling on
"hidden" class
the reason is that the hidden <tr> for stacktrace is a big trouble when you
sort and then expand, or grep some lines.....
so....Welcome to review it.....
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
