Skip to content

Comments

Add api white list and rate limiter for gc#522

Merged
zhoney merged 10 commits intomasterfrom
gc_rate_limit
May 31, 2019
Merged

Add api white list and rate limiter for gc#522
zhoney merged 10 commits intomasterfrom
gc_rate_limit

Conversation

@Linary
Copy link
Contributor

@Linary Linary commented May 23, 2019

Change-Id: Ia7ac5dc5d7d91676701738f2a0ae5073f9ceb01e

@codecov
Copy link

codecov bot commented May 23, 2019

Codecov Report

Merging #522 into master will increase coverage by 0.09%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #522      +/-   ##
============================================
+ Coverage     69.98%   70.07%   +0.09%     
- Complexity     3400     3409       +9     
============================================
  Files           216      216              
  Lines         16500    16503       +3     
  Branches       2359     2360       +1     
============================================
+ Hits          11547    11564      +17     
+ Misses         3692     3681      -11     
+ Partials       1261     1258       -3
Impacted Files Coverage Δ Complexity Δ
...java/com/baidu/hugegraph/type/define/DataType.java 92.53% <100%> (+28.47%) 34 <9> (+11) ⬆️
...n/java/com/baidu/hugegraph/task/TaskScheduler.java 51.45% <0%> (-1.25%) 26% <0%> (-1%)
...va/com/baidu/hugegraph/backend/cache/RamCache.java 72.15% <0%> (-0.4%) 44% <0%> (-1%)
...a/com/baidu/hugegraph/backend/query/Condition.java 64.55% <0%> (ø) 23% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 17008be...5ab35f8. Read the comment docs.

@Linary Linary force-pushed the gc_rate_limit branch 2 times, most recently from aa734b5 to 8364805 Compare May 25, 2019 10:09
Linary added 2 commits May 28, 2019 10:02
Change-Id: Ia7ac5dc5d7d91676701738f2a0ae5073f9ceb01e
Change-Id: Id16eef8f9c4b0b65689d64cb7a44beead2e0cc41
@Linary Linary requested review from javeme and zhoney May 28, 2019 03:21
@Linary Linary force-pushed the gc_rate_limit branch 2 times, most recently from a1a2948 to fdc4879 Compare May 28, 2019 11:44
Change-Id: I9e36d146f2c9463754914f6b2868416254efe371
Change-Id: I7b2429217131e460c08da52a67a1356d670522d9
zhoney
zhoney previously approved these changes May 29, 2019
Change-Id: I4f0bc33115bf50486f9996e287cd4cc9a5edb268
Change-Id: I3e4a28292fcd39cd463d9e9387643e9be1d784cd
Linary added 3 commits May 30, 2019 16:49
Change-Id: I7224a14eb6241f12d459e6263ff47acf93a59d2d
Change-Id: I9ce487c6359ef1a4ed188c8638a7b3812ac27c4d
Change-Id: I7ccd32b2afc16457caf73aae2073c5f23d3bf3fb
Change-Id: I1be0bd5592ff21f7d5914b1042d599816f6020d0
@zhoney zhoney merged commit abebfac into master May 31, 2019
@zhoney zhoney deleted the gc_rate_limit branch May 31, 2019 09:06
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