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
WELD-1866 Probe - config property to skip monitoring of certain events #854
Conversation
mkouba
commented
Feb 11, 2015
- add timestamp to EventInfo
- add timestamp to EventInfo
Triggering build using a merge of 1d901d1 on branch master: |
Build 482 is now running using a merge of 1d901d1 on branch master: |
Build 482 outcome was ABORTED using a merge of 1d901d1 on branch master: |
3 similar comments
Build 482 outcome was ABORTED using a merge of 1d901d1 on branch master: |
Build 482 outcome was ABORTED using a merge of 1d901d1 on branch master: |
Build 482 outcome was ABORTED using a merge of 1d901d1 on branch master: |
retest this please |
Triggering build using a merge of 1d901d1 on branch master: |
Build 487 is now running using a merge of 1d901d1 on branch master: |
Build 487 outcome was SUCCESS using a merge of 1d901d1 on branch master: |
data.data.forEach(function(event) { | ||
event['tsShort'] = moment(event['ts']).format( | ||
'HH:mm:ss'); | ||
event['tsLong'] = moment(event['ts']).format( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can a user switch to tsLong format somehow? I am thinking that having .SSS even in the short version could be useful as many events could be fired within the same second.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tsLong is a title of the span tag which contains tsShort value. But showing millis in the short form is a good idea.
Merged with the time format changed. Thanks! |
Build 487 outcome was SUCCESS using a merge of 1d901d1 on branch master: |