-
Notifications
You must be signed in to change notification settings - Fork 156
Merge DataPlatform Java Client #612
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
Closed
Closed
Changes from all commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
0535126
Initial Data Platform rebranding
srgg 217eb34
Adds initial support of CoverangePlan command and support of coverCo…
srgg f5fa340
Adds HostAndPort.getPortOrDefault()
srgg aeb0988
Makes CoverageEntry serializable
srgg 3a76c34
Fix BinIndexQuery type inconsistency
srgg eec7107
the experimental try to avoid necessarily of key range values for the…
srgg 48b6930
CoveragePlan related code was updated according the latest riak_pb, i…
srgg 091f96c
Java 1.7 compatibility issue
srgg 2511a49
Removes unused/commented code
srgg b3cc4b2
Rename coverContext => coverageContext
srgg f57107d
Fix 2i range local reads
srgg 05c044e
Bumped pom to 1.0.0.beta2
srgg 844d768
LICENSE
srgg 209a8a2
Adding me as a joint author
srgg b6e7250
Bumped pom to 2.0.3
srgg 066f3c0
Added gradle-wrapper.jar
srgg 68d920b
Bumped versions to dataplatform-riak-client-1.0.0
srgg 4596f30
Added com.google.protobuf to the shaded artifact
srgg e1c44d7
Bumped versions to dataplatform-riak-client-1.1.0.beta1
srgg 7b9426d
In order to reuse setupIndexTestData() it was moved to ITestBase
srgg 1b1cd00
Added FullBucketRead command that supports streaming values
srgg ec06e77
RpbIndexBodyResp support was added to SecondaryIndexQueryOperation
srgg c153410
bump versions to 1.1.0.beta2
srgg 5153023
Bump version to 1.1.0
jbrisbin 5e5c970
Update Travis build for publishing to Artifactory
jbrisbin 0edca82
Add secure resolver config.
jbrisbin 466a3b8
Tweak to build file.
jbrisbin 6592c1e
java source compatibility version changed to 1.7
vikua 8f9f167
Bump version to 1.2.0-beta1, add Artifactory publish
jbrisbin fafa05e
Added Basho Artifactory repo.
jbrisbin d3b420f
Changed version to 1.2.0-SNAPSHOT
jbrisbin f365d31
Coverage entry verification to be sure that CoveragePlan doesn't have…
srgg 1611931
Bump version to 1.3.0-beta1
srgg 062c956
[SPARK-78] Fix for argument type in CoveragePlanOperation.AbstractBui…
oleksii-suprun 5afa6d0
Tests added to check RiakKV behavior in case of unavailability of sui…
AleksandrPavlenko bc22afb
[SPARK-119] Add RiakTS Coverage Plan support to dataplatform java client
zkhadikova 5b9c07c
Bump version to 1.5.0-SNAPSHOT
oleksii-suprun 78b0224
Bump version to 1.5.1-SNAPSHOT
oleksii-suprun 689148f
Automatic RiakTS Test table creation added to integration tests
srgg ebce61a
Return riak client branding back in pom.xml
srgg 5bd8354
Merge branch 'develop' into merge-dpclient
alexmoore bd6cf99
Fix merge issues
alexmoore 2396fcf
Add test for coverage context in TTBCodec
alexmoore 5fa483b
Merge pull request #621 from basho/merge-dpclient-ts1.3
srgg c125edc
[SPARK-165] Adjust merged riak-java-client according to comments
oleksii-suprun File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,3 +6,4 @@ | |
|
|
||
| *.png binary | ||
| *.jpg binary | ||
| *.jar binary | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,7 +3,6 @@ lib | |
| classes | ||
| target | ||
| .settings | ||
| *.jar | ||
| .classpath | ||
| .project | ||
| .idea | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,10 +1,3 @@ | ||
| language: java | ||
| script: mvn -Pitest -DargLine="-Dcom.basho.riak.search=true -Dcom.basho.riak.2i=true" clean verify | ||
| notifications: | ||
| # Email notifications are disabled to not annoy anybody. | ||
| # See http://about.travis-ci.org/docs/user/build-configuration/ to learn more | ||
| # about configuring notification recipients and more. | ||
| webhooks: http://basho-engbot.herokuapp.com/travis?key=a8f477c8cdaabc8118d2a650281ff4a71eff875f | ||
| email: false | ||
| services: | ||
| - riak | ||
| script: | ||
| - "mvn deploy -DskipTests" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -159,6 +159,7 @@ | |
| </plugins> | ||
| </build> | ||
| </profile> | ||
|
|
||
| <profile> | ||
| <id>release</id> | ||
| <activation> | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We include riak_pb as a submodule now, so we can remove this step once that develop is merged back to this branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll rebase from the latest develop, that should resolve this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@srgg I have a branch that does that - There were some changes to the TSQuery command since we switched to term-to-binary support. https://github.com/basho/riak-java-client/tree/merge-dpclient-ts1.3
We need a test around using a coverage context with a TS query request. I need to see if I wrote this line correctly:
riak-java-client/src/main/java/com/basho/riak/client/core/codec/TermToBinaryCodec.java
Line 82 in bd6cf99
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We would need to add a test similar to this one, that adds a context, to make sure it encodes correctly: https://github.com/basho/riak-java-client/blob/merge-dpclient-ts1.3/src/test/java/com/basho/riak/client/core/codec/TermToBinaryCodecTest.java#L133
I think I added everything else in that branch to add the coverage context to the TTB TS Queries though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After merging1.3 branch into this one, @aleksey-suprun can prove this conversion by running (or extending, if needed) our Full Bucket Read Test