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.

Sign up for GitHub

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

Jump to bottom

Add time filtering when querying agents for Realtime data #1318

Closed
Xylus opened this issue Dec 4, 2015 · 2 comments · Fixed by #1497
Closed

Add time filtering when querying agents for Realtime data #1318

Xylus opened this issue Dec 4, 2015 · 2 comments · Fixed by #1497
Assignees
@Xylus
Labels
enhancement
Milestone
1.5.2

Comments

@Xylus
Copy link
Contributor

Xylus commented Dec 4, 2015

When querying for realtime data such as the active thread chart, servers that are no longer in use are of no interest. These servers are currently displayed as NOT_FOUND and will continue to be displayed until the TTL value set in HBase.

It might be a good idea to set a reasonable time buffer (perhaps a week?) when querying for these agents so that servers that were down for longer than a week does not show up in the active thread chart and confuse everyone.
@Xylus Xylus self-assigned this Dec 4, 2015
@Xylus Xylus added this to the 1.5.2 milestone Dec 4, 2015
Xylus added a commit to Xylus/pinpoint that referenced this issue Dec 7, 2015
@Xylus
pinpoint-apm#1318 Exclude agents that have been disconnected for too …
ded07ec 
…long
@Xylus Xylus mentioned this issue Dec 7, 2015
Merged
Xylus added a commit to Xylus/pinpoint that referenced this issue Dec 7, 2015
@Xylus
pinpoint-apm#1318 Exclude agents that have been disconnected for too …
8c05199 
…long
@Xylus
Copy link
Contributor Author

Xylus commented Dec 8, 2015

Api change affects transaction list queries - reopening issue.

@Xylus Xylus reopened this Dec 8, 2015
Xylus added a commit to Xylus/pinpoint that referenced this issue Dec 8, 2015
@Xylus
pinpoint-apm#1318 Remove negative timeDiff check as timestamp may be …
0e5176e 
…less than 0
Xylus added a commit that referenced this issue Dec 8, 2015
@Xylus
Merge pull request #1333 from Xylus/bugfix/issue-1318
b18000a 
#1318 Remove negative timeDiff check as timestamp may be less than 0
@Xylus Xylus closed this as completed Dec 8, 2015
@Xylus
Copy link
Contributor Author

Xylus commented Jan 28, 2016

Reopening issue to add configuration option for the duration determining agent inactivity.

@Xylus Xylus reopened this Jan 28, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Jan 28, 2016
@Xylus
pinpoint-apm#1318 Make active thread agent query duration configurable
4a3beef
@Xylus Xylus mentioned this issue Jan 28, 2016
Merged
@Xylus Xylus mentioned this issue Jan 10, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Xylus Xylus

Labels
enhancement
Projects
None yet
Milestone
1.5.2
Development

Successfully merging a pull request may close this issue.

Make active thread agent query duration configurable Xylus/pinpoint
1 participant
@Xylus