restatedev / sdk-java

Restate SDK for JVM Languages
https://restate.dev
MIT License
19 stars 4 forks source link

Remove usage of jackson in sdk-common ingress client #328

Open slinkydeveloper opened 1 month ago

slinkydeveloper commented 1 month ago

Remove the other usage of jackson in sdk-common, depends on https://github.com/restatedev/restate/pull/1542

github-actions[bot] commented 1 month ago

Test Results

 48 files  ±0   48 suites  ±0   2m 55s :stopwatch: -12s 738 tests ±0  716 :white_check_mark:  - 13  8 :zzz:  - 1  14 :x: +14  738 runs   - 1  716 :white_check_mark:  - 14  8 :zzz:  - 1  14 :x: +14 

For more details on these failures, see this check.

Results for commit 5219f3f1. ± Comparison against base commit 2b5d8e42.

This pull request removes 7 and adds 7 tests. Note that renamed tests count towards both. ``` dev.restate.e2e.NodeWorkflowAPITest ‑ Workflow cannot be submitted more than once dev.restate.sdk.common.CoreSerdesTest ‑ Byte: -111 dev.restate.sdk.common.CoreSerdesTest ‑ Double: 0.5184783207844548 dev.restate.sdk.common.CoreSerdesTest ‑ Float: 0.62707436 dev.restate.sdk.common.CoreSerdesTest ‑ Integer: 1461953937 dev.restate.sdk.common.CoreSerdesTest ‑ Long: -4178086973975213529 dev.restate.sdk.common.CoreSerdesTest ‑ Short: 13800 ``` ``` dev.restate.sdk.common.CoreSerdesTest ‑ Byte: -89 dev.restate.sdk.common.CoreSerdesTest ‑ Double: 0.9046595808179864 dev.restate.sdk.common.CoreSerdesTest ‑ Float: 0.26799035 dev.restate.sdk.common.CoreSerdesTest ‑ Integer: 1669200667 dev.restate.sdk.common.CoreSerdesTest ‑ Long: 5344576980110012853 dev.restate.sdk.common.CoreSerdesTest ‑ Short: 11065 dev.restate.sdk.serde.jackson.JacksonSerdesTest ‑ [Till, Francesco] ```

:recycle: This comment has been updated with latest results.