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.

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

Changed the metric unit of latency from ms to us #267

Merged
merged 3 commits into from
Apr 12, 2019
Merged

Changed the metric unit of latency from ms to us #267

merged 3 commits into from
Apr 12, 2019

Conversation

dutor
Copy link
Contributor

@dutor dutor commented Apr 11, 2019

Millisecond is too coarse for latency.

@nebula-community-bot
Copy link
Member

Unit testing passed.

@nebula-community-bot
Copy link
Member

Unit testing passed.

Copy link
Contributor

@dangleptr dangleptr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@nebula-community-bot
Copy link
Member

Unit testing passed.

@dutor dutor merged commit 85cfd1c into vesoft-inc:master Apr 12, 2019
@dutor dutor deleted the latency-in-us branch April 12, 2019 06:36
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Feb 16, 2020
* Changed the metric unit of latency from ms to us

* Rebase and address @dangleptr's comment
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
* Changed the metric unit of latency from ms to us

* Rebase and address @dangleptr's comment
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Mar 21, 2022
* stash

* commit IndexEdge/VertexScanNode

* commit IndexSelectionNode

* commit Projection/Dedup Node

* commit IndexLimitNode

* stash

* stash

* stash

* stash

* add index int test

* add double/string1 test

* add string2/string3 test

* finish index_scan_node unittest

* add index node test

* pass lookupindex test

* pass ttl test

* pass all unittest

* remove debug log

* fix bug

* fix bug

* fix bug

* clear file

* clear some useless code

* add comments for IndexNode

* add comments to IndexScanNode

* ad comment to Selection/Projection/DedupNode

* fix some init bug

* fix bug to support geo

* converge qualified strategy define together

* address comments

* address some comments; Modify IndexNode::next return Type

* fix bug

* add some comments

* Add blank lines between function definitions

* fix compile error

* modify new file license

* Modify test to avoid the bug mentioned in issue3191

* modify license error

Co-authored-by: hs.zhang <22708345+cangfengzhs@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants