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

change limit type long to int #349

Merged
merged 2 commits into from
Nov 1, 2022
Merged

change limit type long to int #349

merged 2 commits into from
Nov 1, 2022

Conversation

zyxxoo
Copy link
Contributor

@zyxxoo zyxxoo commented Oct 30, 2022

No description provided.

@zyxxoo zyxxoo requested review from imbajin and javeme October 30, 2022 12:45
@zyxxoo zyxxoo changed the title chage limit type long to int change limit type long to int Oct 30, 2022
Copy link
Member

@imbajin imbajin left a comment

Choose a reason for hiding this comment

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

seems I try to change it in server formerly, but suspend for some reason.... @javeme

Update: merge it after the CI PR merged

@zyxxoo zyxxoo requested a review from imbajin October 30, 2022 14:57
@codecov
Copy link

codecov bot commented Nov 1, 2022

Codecov Report

Merging #349 (be684ff) into master (6d82b15) will decrease coverage by 28.51%.
The diff coverage is 71.42%.

@@              Coverage Diff              @@
##             master     #349       +/-   ##
=============================================
- Coverage     67.43%   38.92%   -28.52%     
- Complexity      878      966       +88     
=============================================
  Files            86      242      +156     
  Lines          4029     8111     +4082     
  Branches        477      719      +242     
=============================================
+ Hits           2717     3157      +440     
- Misses         1107     4775     +3668     
+ Partials        205      179       -26     
Impacted Files Coverage Δ
.../baidu/hugegraph/api/traverser/CrosspointsAPI.java 100.00% <ø> (ø)
...om/baidu/hugegraph/api/traverser/KneighborAPI.java 100.00% <ø> (ø)
...ava/com/baidu/hugegraph/api/traverser/KoutAPI.java 100.00% <ø> (ø)
...va/com/baidu/hugegraph/api/traverser/PathsAPI.java 100.00% <ø> (ø)
...baidu/hugegraph/api/traverser/PersonalRankAPI.java 85.96% <ø> (ø)
...ava/com/baidu/hugegraph/api/traverser/RaysAPI.java 100.00% <ø> (ø)
...va/com/baidu/hugegraph/api/traverser/RingsAPI.java 100.00% <ø> (ø)
...aidu/hugegraph/api/traverser/SameNeighborsAPI.java 100.00% <ø> (ø)
...aph/api/traverser/SingleSourceShortestPathAPI.java 100.00% <ø> (ø)
...m/baidu/hugegraph/api/traverser/TraversersAPI.java 88.00% <ø> (ø)
... and 246 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@javeme javeme merged commit 12a490e into master Nov 1, 2022
@javeme javeme deleted the zy_dev branch November 1, 2022 12:30
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