Merge master by artursouza · Pull Request #460 · dapr/java-sdk

artursouza · 2021-01-27T23:59:15Z

Description

Merge master into release-1.0

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: endgame

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

Code compiles correctly
Created/updated tests
Extended the documentation

Signed-off-by: Arghya Sadhu <arghya88@gmail.com>

Co-authored-by: Mukundan Sundararajan <musundar@microsoft.com> Co-authored-by: Artur Souza <artursouza.ms@outlook.com>

)

* add metadata in get bulk state request * rollback DaprClientGrpcTest which should be comitted in another PR * fix compile error after merge * update unit test for metadata in getBulkState() * re-trigger build again * add test for withMetadata() Co-authored-by: Mukundan Sundararajan <musundar@microsoft.com>

Signed-off-by: Arghya Sadhu <arghya88@gmail.com> Co-authored-by: Artur Souza <artursouza.ms@outlook.com>

* Fix input encoding and error parsing for HTTP calls. * Adds tests for special char in HTTP call. * Rename path to pathSegments

…ontent-type in PubSub over GRPC (#431)

* Fix blocking calls for gRPC even though using Reactor * delete unused wrap method * add: throw exception if fatal * Add "NoHotMono" unit tests for DaprClientGrpc Co-authored-by: 刘禅 <zuojie@alibaba-inc.com> Co-authored-by: Artur Souza <artursouza.ms@outlook.com>

* Fix handling of metadata, headers. Removes etag from getState(). * Update CLI and runtime. * Fix ITs.

* Rename InvokeServiceRequest to InvokeMethodRequest. * Organize imports.

* Add support deserialization of generic types per #293 * Improving test coverage

* Makes service method API invocations over HTTP by default. * Add logic to avoid building managedchannel in case of http API for actor runtime.

* Use DaprChannel instead of closeable ActorProxyBuilder. * Refactor Actor channel into DaprChannel and out of ActorProxyBuilder. * Rename DaprChannel to ActorClient and make it a DaprClient. * Make ActorClient not implement DaprClient but implement same method. * Update sdk-actors/src/main/java/io/dapr/actors/client/ActorProxyBuilder.java Co-authored-by: Mukundan Sundararajan <musundar@microsoft.com> * Update sdk-actors/src/test/java/io/dapr/actors/client/ActorProxyBuilderTest.java Co-authored-by: Mukundan Sundararajan <musundar@microsoft.com> * Rename ActorProxyForTestsImpl to ActorProxyImplForTests Co-authored-by: Mukundan Sundararajan <musundar@microsoft.com>

Adding GetBulkSecret API.

Unit tests for nullable-etag (#442)

…handling (#458) * Updating state management example to include etag mismatch exception scenario. Dapr runtime grpc exception handling was recently updated to return aborted error code on etage mismatch for mutations, the example is updated to reflect that the runtime updates will be propagated through the java sdk. * Addressed feedback to merge block call with api call. * Updating readme with block() call updates, missed in previous commit

* Adding Dapr Spring Boot Starter. * Move Dapr Spring Boot Starter into existing sdk-springboot project. Co-authored-by: Darren Wang <darren_wang@outlook.com>

codecov · 2021-01-28T00:28:47Z

Codecov Report

Merging #460 (141726d) into release-1.0 (8c79d7c) will decrease coverage by 0.34%.
The diff coverage is 83.09%.

@@                Coverage Diff                @@
##             release-1.0     #460      +/-   ##
=================================================
- Coverage          83.26%   82.92%   -0.35%     
- Complexity           835      931      +96     
=================================================
  Files                 72       79       +7     
  Lines               2486     2717     +231     
  Branches             255      269      +14     
=================================================
+ Hits                2070     2253     +183     
- Misses               293      331      +38     
- Partials             123      133      +10

Impacted Files	Coverage Δ	Complexity Δ
...ain/java/io/dapr/actors/client/DaprGrpcClient.java	`87.50% <ø> (ø)`	`6.00 <0.00> (ø)`
...main/java/io/dapr/actors/runtime/ActorRuntime.java	`56.16% <0.00%> (ø)`	`20.00 <0.00> (ø)`
...in/java/io/dapr/actors/runtime/DaprGrpcClient.java	`95.50% <ø> (ø)`	`20.00 <0.00> (ø)`
.../dapr/client/domain/DeleteStateRequestBuilder.java	`85.00% <ø> (ø)`	`5.00 <0.00> (ø)`
sdk/src/main/java/io/dapr/client/domain/State.java	`74.39% <ø> (ø)`	`20.00 <0.00> (+1.00)`
...rc/main/java/io/dapr/client/DaprClientBuilder.java	`68.42% <58.82%> (+7.13%)`	`11.00 <3.00> (+3.00)`
...c/main/java/io/dapr/actors/client/ActorClient.java	`59.09% <59.09%> (ø)`	`6.00 <6.00> (?)`
sdk/src/main/java/io/dapr/config/Properties.java	`77.77% <71.42%> (-7.94%)`	`1.00 <0.00> (ø)`
.../src/main/java/io/dapr/client/DaprClientProxy.java	`71.92% <71.92%> (ø)`	`35.00 <35.00> (?)`
...rc/main/java/io/dapr/exceptions/DaprException.java	`73.46% <72.72%> (-2.73%)`	`17.00 <6.00> (+2.00)`	⬇️
... and 43 more

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 8c79d7c...a42120f. Read the comment docs.

artursouza and others added 29 commits December 15, 2020 23:34

Upgrade the version to 1.0.0-SNAPSHOT (#413)

de4c37e

Rename methods and parameters for consistency across SDKs (#410)

11d9e3b

Signed-off-by: Arghya Sadhu <arghya88@gmail.com>

Split components dir for samples + other fixes. (#417)

b0f1a44

Update zipkin detail page for tracing sample. (#423)

7f1ce0b

set etag to be null if it's empty (#414)

a35e414

Signed-off-by: Arghya Sadhu <arghya88@gmail.com>

Rename ActorProxy invoke to invokeMethod. (#420)

3b51b5b

Co-authored-by: Mukundan Sundararajan <musundar@microsoft.com> Co-authored-by: Artur Souza <artursouza.ms@outlook.com>

Throws IllegalArgumentException without wrapping as DaprException. (#426

08f133d

)

refactor to move duplicate code to method (#429)

4c1773c

Signed-off-by: Arghya Sadhu <arghya88@gmail.com> Co-authored-by: Artur Souza <artursouza.ms@outlook.com>

Fix input encoding and error parsing for HTTP calls. (#428)

2714402

* Fix input encoding and error parsing for HTTP calls. * Adds tests for special char in HTTP call. * Rename path to pathSegments

Support TTL in PubSub. Fixes + refactor of CloudEvent. Fix handling c…

1a6675a

…ontent-type in PubSub over GRPC (#431)

Adds method to wait for Dapr sidecar. (#433)

6a44d51

Refactor tracing demo to show A->B->C call stack. (#436)

9e13586

Fix handling of metadata, headers. Removes etag from getState. (#440)

019dee9

* Fix handling of metadata, headers. Removes etag from getState(). * Update CLI and runtime. * Fix ITs.

Rename InvokeServiceRequest to InvokeMethodRequest. (#443)

d43272f

* Rename InvokeServiceRequest to InvokeMethodRequest. * Organize imports.

Support ActorMethod to change method's name. (#448)

446cd17

Makes http call async and allows timeout in Mono. (#445)

81b47f8

Add java.lang.reflect.Type to TypeRef per #331 (#449)

38b899d

* Add support deserialization of generic types per #293 * Improving test coverage

Makes service method API invocations over HTTP by default. (#450)

befed4a

* Makes service method API invocations over HTTP by default. * Add logic to avoid building managedchannel in case of http API for actor runtime.

Adding GetBulkSecret API.

907748f

Merge pull request #452 from artursouza/get_bulk_secrets

233c699

Adding GetBulkSecret API.

Added unit tests for nullable etag #442

c35f0e1

Merge pull request #454 from wcs1only/nullable-etag

faf3bec

Unit tests for nullable-etag (#442)

Add version to Components. (#455)

c5b4cbd

Adds autoconfiguration for springboot apps. (#459)

6c26942

* Adding Dapr Spring Boot Starter. * Move Dapr Spring Boot Starter into existing sdk-springboot project. Co-authored-by: Darren Wang <darren_wang@outlook.com>

Merge branch 'master' into release-1.0

a42120f

mukundansundar approved these changes Jan 28, 2021

View reviewed changes

artursouza merged commit e7ae728 into dapr:release-1.0 Jan 28, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Merge master#460

Merge master#460
artursouza merged 29 commits into
dapr:release-1.0from
artursouza:merge_master

artursouza commented Jan 27, 2021

Uh oh!

codecov Bot commented Jan 28, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

Uh oh!

Conversation

artursouza commented Jan 27, 2021

Description

Issue reference

Checklist

Uh oh!

codecov Bot commented Jan 28, 2021

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants