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.

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

Remove protobuf from the public api of sdk-common #321

Merged
merged 1 commit into from
May 21, 2024
Merged

Conversation

slinkydeveloper
Copy link
Contributor

No description provided.

Copy link
Contributor

github-actions bot commented May 21, 2024

Test Results

 48 files  +34   48 suites  +34   3m 11s ⏱️ + 3m 1s
739 tests +75  730 ✅ +73  9 💤 +2  0 ❌ ±0 
739 runs  +74  730 ✅ +72  9 💤 +2  0 ❌ ±0 

Results for commit 0b4e329. ± Comparison against base commit 1a5817c.

This pull request removes 7 and adds 82 tests. Note that renamed tests count towards both.
dev.restate.sdk.common.CoreSerdesTest ‑ Byte: -101
dev.restate.sdk.common.CoreSerdesTest ‑ Double: 0.786505251599144
dev.restate.sdk.common.CoreSerdesTest ‑ Float: 0.66245365
dev.restate.sdk.common.CoreSerdesTest ‑ Integer: 1077840941
dev.restate.sdk.common.CoreSerdesTest ‑ Long: -144241236709006023
dev.restate.sdk.common.CoreSerdesTest ‑ Short: 6611
dev.restate.sdk.common.CoreSerdesTest ‑ byte[]: [B@256f8274
dev.restate.e2e.JavaAwaitTimeoutTest ‑ Test Awaitable#await(Duration)
dev.restate.e2e.JavaAwakeableTest ‑ generate(IngressClient)
dev.restate.e2e.JavaCancelInvocationTest ‑ cancel blocked invocation on AWAKEABLE
dev.restate.e2e.JavaCancelInvocationTest ‑ cancel blocked invocation on CALL
dev.restate.e2e.JavaCancelInvocationTest ‑ cancel blocked invocation on SLEEP
dev.restate.e2e.JavaCoordinatorWithNodeReceiverServiceToServiceCallTest ‑ synchronousCall(IngressClient)
dev.restate.e2e.JavaErrorsTest ‑ Test calling method that fails terminally
dev.restate.e2e.JavaErrorsTest ‑ Test calling method that fails terminally multiple times
dev.restate.e2e.JavaErrorsTest ‑ Test invocations are retried until they succeed
dev.restate.e2e.JavaErrorsTest ‑ Test propagate failure from another service
…

♻️ This comment has been updated with latest results.

@slinkydeveloper slinkydeveloper merged commit cc52be1 into main May 21, 2024
4 checks passed
@slinkydeveloper slinkydeveloper deleted the issues/311 branch May 21, 2024 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant