8000 useNanoTime option does not seem to be taken into account · Issue #462 · p6spy/p6spy · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

useNanoTime option does not seem to be taken into account #462

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
Open
romainguinot opened this issue Dec 17, 2018 · 3 comments
Open

useNanoTime option does not seem to be taken into account #462

romainguinot opened this issue Dec 17, 2018 · 3 comments
Labels
Defect Help needed

Comments

@romainguinot
Copy link

Hi,

I have seen in issue #295 a config option, useNanoTime, was added.
I have tried it with p6spy 3.8.1, but it does not seem like it is used. The results i get in the log file are almost the same as when not using the option.

Using JDK 1.8.0_121.
I doubt this can be used for truly high precision though for a number of factors, and it is likely to be better suited to investigate queries over the network, which would likely take more than a couple ms including the network RTT.
@typekpb
Copy link
Member
typekpb commented Dec 17, 2018

@romainguinot briefly checked the code, and nano seems to be no more taken into account, let me check deeper and provide the fix (or maybe a reference, where we lost the support)

@typekpb typekpb added the Defect label Dec 17, 2018
@typekpb
Copy link
Member
typekpb commented Jul 21, 2019

We need a volunteer for implementation. I have no time for that, sorry.

peshek-of-rattay added a commit to peshek-of-rattay/p6spy that referenced this issue Apr 8, 2021
@peshek-of-rattay
useNanoTime option is not taken into account (p6spy#462)
5c3e0f3
@pesh 6F27 ek-of-rattay
Copy link

@typekpb, I've fixed the issue. See the pull request #529.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Defect Help needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@typekpb @romainguinot @peshek-of-rattay
0