Raccoon Rest/Http Java Client #51
Merged
Conversation
This file contains 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
Member
AkbaraliShaikh
commented
Nov 9, 2022
•
AkbaraliShaikh
commented
- Implemented the Rest Java client for raccoon
- Implemented http request retry handler
- Added the JSON/PROTO serialisers
- Added the Wire formatter (Json/Proto)
- Added the unit tests
- Added the WireMock Server for testing the http requests
AkbaraliShaikh
assigned AkbaraliShaikh, ravisuhag and chakravarthyvp and unassigned ravisuhag and chakravarthyvp
AkbaraliShaikh
changed the title
added serializer for json and proto add serializer and deserializer for raccoon request added unit tests added mock server test for rest client feat: raccoon proto generated code feat: added custom exception types feat: added retry logic feat: added test files feat: custom exception feat: buf proto generate feat: added the java 1.8 version compatibility
AkbaraliShaikh
force-pushed
the
raccoon-java-client
branch
from
0714504
to
9e11b48
Compare
clients/java/src/main/java/io/odpf/raccoon/RaccoonClientFactory.java
Outdated
Show resolved
Hide resolved
feat: compile version set to 1.8 feat: compile version set to 1.8 test: testing and fixing the pipeline test: testing and fixing the pipeline test: testing and fixing the pipeline test: testing and fixing the pipeline test: testing and fixing the pipeline test: testing and fixing the pipeline
AkbaraliShaikh
force-pushed
the
raccoon-java-client
branch
from
7ab4190
to
188136f
Compare
clients/java/src/main/java/io/odpf/raccoon/client/RaccoonClient.java
Outdated
Show resolved
Hide resolved
clients/java/src/main/java/io/odpf/raccoon/client/rest/RestClient.java
Outdated
Show resolved
Hide resolved
clients/java/src/main/java/io/odpf/raccoon/client/rest/RestClient.java
Outdated
Show resolved
Hide resolved
doc: updated the README file
chakravarthyvp
approved these changes
Dec 1, 2022
|
@ravisuhag can you please check the publish action? the Jar is not available for import. |
|
@AkbaraliShaikh checking. meanwhile can you please bring the version to same as raccoon in another PR that is raised. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.