Github user d2r commented on the pull request:
https://github.com/apache/storm/pull/388#issuecomment-102503828
> @d2r Thanks, and that is another thing - why do we have log link on port
number?
> That does not make any sense at all.The truncated text is where the user
expects a error log link on.
@rekhajoshm , sorry, somehow I did not get notified of this post.
The worker log link on the port number corresponds to the way the worker
log link is designed into the Executors table on the Component page. The two
are thus consistent. But that does not mean it makes more sense than having
the link to the worker log somewhere else. You have raised a good point.
It turns out that this change breaks the red-coloring of errors reported
less than 5 minutes in age (since it makes the entire text a blue link), so we
need to revert this change for now.
I am -1, since it breaks functionality.
---
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.
---
