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

First cut at updating reactivex-grpc dependencies #335

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

scottslewis
Copy link
Collaborator

First cut at updating reactivex-grpc dependencies:

    <grpc.version>1.71.0</grpc.version> was 1.54.0
    <protoc.version>4.30.0</protoc.version> was 3.22.2
	<reactor.version>3.7.3</reactor.version> was 3.5.4
    <rx3java.version>3.1.10</rx3java.version> was 3.1.10

        <grpc.version>1.71.0</grpc.version> was 1.54.0
        <protoc.version>4.30.0</protoc.version> was 3.22.2
		<reactor.version>3.7.3</reactor.version> was 3.5.4
        <rx3java.version>3.1.10</rx3java.version> was 3.1.10

Signed-off-by: Scott Lewis <[email protected]>
@scottslewis scottslewis requested a review from rmichela March 6, 2025 00:56
@scottslewis scottslewis self-assigned this Mar 6, 2025
@scottslewis
Copy link
Collaborator Author

@rmichela

Here is an update of some of the reactivex-grpc dependencies. In my environment, this all compiles except for the benchmark project.

Most of the tests succeed, but a number of them are failing with message like the following:

java.lang.IllegalStateException: ReactorClientStreamObserverAndPublisher allows only a single Subscriber
at com.salesforce.reactivegrpc.common.AbstractStreamObserverAndPublisher.subscribe(AbstractStreamObserverAndPublisher.
java:360)
...

Can you advise on this? Attached is the entire output of the build and test run.

mvnoutput.txt

@scottslewis
Copy link
Collaborator Author

@rmichela any chance of getting a release done with these dependency updates?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant