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

chore: Merge java client #964

Merged
merged 127 commits into from
May 9, 2022
Merged

chore: Merge java client #964

merged 127 commits into from
May 9, 2022

Conversation

cauchy1988
Copy link
Contributor

What problem does this PR solve?

merge pegasus java client into incubator-pegasus

details can be seen in the issue : #945

What is changed and how does it work?

only add the java client repo

Checklist

Tests
  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code
Code changes
  • Has exported function/method change
  • Has exported variable/fields change
  • Has interface methods change
  • Has persistent data change
Side effects
  • Possible performance regression
  • Increased code complexity
  • Breaking backward compatibility
Related changes
  • Need to cherry-pick to the release branch
  • Need to update the documentation
  • Need to be included in the release note

shengofsun and others added 30 commits March 10, 2018 11:29
@shengofsun
First commit
ba4dc3c
@shengofsun
add java client source code
c6bb6ff 
Summary: Ref T9266

Test Plan: N/A

Reviewers: qinzuoyan, cailiuyang, heyuchen, wutao1, laiyingchun

Reviewed By: qinzuoyan

Subscribers: #pegasus

Maniphest Tasks: T9266

Differential Revision: https://phabricator.d.xiaomi.net/D86668
@shengofsun
rpc: fix delay overflow, explictly close socket when close, fix test
46a3343 
Summary: Ref T10114

Test Plan: N/A

Reviewers: qinzuoyan, cailiuyang

Reviewed By: qinzuoyan

Subscribers: #pegasus

Maniphest Tasks: T10114

Differential Revision: https://phabricator.d.xiaomi.net/D87645
@shengofsun
fix ut: MetricsPoolTest mistakenly relying on the output order of jso…
15be374 
…n obj

Summary: Ref T10114

Test Plan: N/A

Reviewers: qinzuoyan, cailiuyang

Reviewed By: qinzuoyan

Subscribers: #pegasus

Maniphest Tasks: T10114

Differential Revision: https://phabricator.d.xiaomi.net/D87746
@qinzuoyan
add reverse option in MultiGetOptions to support search in reverse di…
cdba5e4 
…rection for multiGet

Summary: Ref T10566

Test Plan: N/A

Reviewers: sunweijie, cailiuyang

Reviewed By: sunweijie

Subscribers: #pegasus

Maniphest Tasks: T10566

Differential Revision: https://phabricator.d.xiaomi.net/D87870
@qinzuoyan
fix logging
7fbde17 
Summary: Ref T10114

Test Plan: N/A

Reviewers: sunweijie, cailiuyang, laiyingchun

Reviewed By: laiyingchun

Subscribers: #pegasus

Maniphest Tasks: T10114

Differential Revision: https://phabricator.d.xiaomi.net/D88948
@qinzuoyan
Bump version to 1.9.SNAPSHOT-thrift-0.11.0-inlined
566b68a
@acelyc111 @shengofsun
idl: add namespace for python
e8236e7
@qinzuoyan
add scan test; change default scan batch size (apache#4)
5924f49
@qinzuoyan
support incr (apache#7)
757d789
@qinzuoyan
Bump version to 1.10.SNAPSHOT-thrift-0.11.0-inlined
78e879f
@qinzuoyan
add checkAndSet and compareExchange interface (apache#8)
d941b55
@acelyc111
Bump version to 1.11.SNAPSHOT-thrift-0.11.0-inlined
e2cb668
@shengofsun
rpc: use nio socket channel of netty (apache#9)
987796a
@qinzuoyan
make fields of CheckAndSetResult/CompareExchangeResult public (apache#10
bdf69da 
)
table: bugfix for hashkey filtering of unorderedScan (apache#12)
1414e6c
@shengofsun @qinzuoyan
rpc: add message filter to support anandon message easily (apache#13)
5822933
@qinzuoyan
add check_and_mutate() (apache#11)
7927b4b
@hycdong
Bump version to 1.12.SNAPSHOT-thrift-0.11.0-inlined
171e98c
@qinzuoyan
Mutations: constructor public (apache#14)
9d0b96c
Mutations: make all member methods public (apache#15)
bac5d3a
@shengofsun @qinzuoyan
rpc: add some comments (apache#16)
e06bd86
@qinzuoyan
interface: incr support reset ttl (apache#17)
a4e45c9
@qinzuoyan
tools: fix Toollet static{} (apache#18)
cf7893e
@qinzuoyan
logging: validate table name and print log for problem diagnosis (apa…
c8d0bb2 
…che#20)
@qinzuoyan
tools: add zstd compression support (apache#21)
e28961d
@qinzuoyan
add copy_data operator in PegasusCli (apache#23)
5b9a69a
@qinzuoyan
set client_timeout in header (apache#24)
53e1908
@qinzuoyan
support get forward address from configuration_query_by_index_response (
5c260ce 
apache#27)
@qinzuoyan
re-implement getting forward address from configuration_query_by_inde…
920217a 
…x_response (apache#28)
@cauchy1988 cauchy1988 changed the title Merge java client chore: Merge java client May 7, 2022
@cauchy1988 cauchy1988 force-pushed the merge-java-client branch 4 times, most recently from 6c421ee to c2ded94 Compare May 7, 2022 12:23
@acelyc111
Copy link
Member

acelyc111 commented May 9, 2022 •
edited
Loading

I checked the following changes:

  • All files missing APLv2 license header have been added.
  • Package com.xiaomi.infra.pegasus has been rename to org.apache.pegasus

@acelyc111
Copy link
Member

acelyc111 commented May 9, 2022 •
edited
Loading

Java client link in README.md should be updated.

@acelyc111
Copy link
Member

Unit test action [1] should also be migrated to this repo.

[1] https://github.com/XiaoMi/pegasus-java-client/blob/master/.github/workflows/ci-pr.yaml

@cauchy1988
Copy link
Contributor Author

Unit test action [1] should also be migrated to this repo.

[1] https://github.com/XiaoMi/pegasus-java-client/blob/master/.github/workflows/ci-pr.yaml

i have already added， it just be renamed to a new file ”test_java-client.yml“

acelyc111
acelyc111 reviewed May 9, 2022
View reviewed changes
java-client/README.md Outdated Show resolved Hide resolved
@acelyc111
Copy link
Member

Unit test action [1] should also be migrated to this repo.
[1] https://github.com/XiaoMi/pegasus-java-client/blob/master/.github/workflows/ci-pr.yaml

i have already added， it just be renamed to a new file ”test_java-client.yml“

OK, I saw it.

@cauchy1988 cauchy1988 force-pushed the merge-java-client branch from c2ded94 to 418d620 Compare May 9, 2022 04:13
acelyc111
acelyc111 reviewed May 9, 2022
View reviewed changes
java-client/README.md Outdated Show resolved Hide resolved
@cauchy1988 cauchy1988 force-pushed the merge-java-client branch from 418d620 to 6f2848c Compare May 9, 2022 04:24
acelyc111
acelyc111 previously approved these changes May 9, 2022
View reviewed changes
@cauchy1988 cauchy1988 force-pushed the merge-java-client branch from 6f2848c to c1f1538 Compare May 9, 2022 06:15
acelyc111
acelyc111 approved these changes May 9, 2022
View reviewed changes
Copy link
Member

@acelyc111 acelyc111 left a comment

Choose a reason for hiding this comment

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

LGTM

neverchanje
neverchanje approved these changes May 9, 2022
View reviewed changes
Copy link
Contributor

@neverchanje neverchanje left a comment

Choose a reason for hiding this comment

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

lgtm

@acelyc111 acelyc111 merged commit fbed9a1 into apache:master May 9, 2022
acelyc111 pushed a commit that referenced this pull request Jun 23, 2022
@hycdong
feat(bulk_load): add unhealthy partition check (#964)
cba16ed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acelyc111 acelyc111 acelyc111 approved these changes

@neverchanje neverchanje neverchanje approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@cauchy1988 @acelyc111 @neverchanje @shengofsun @qinzuoyan @hycdong @foreverneverer @levy5307 @ZhongChaoqiang