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

Upgrade to Reactor Dysprosium #670

Closed
rstoyanchev opened this issue Jul 25, 2019 · 1 comment · Fixed by #671
Closed

Upgrade to Reactor Dysprosium #670

rstoyanchev opened this issue Jul 25, 2019 · 1 comment · Fixed by #671

Comments

@rstoyanchev
Copy link
Contributor

When trying to upgrade spring-messaging to rsocket-core version 1.0.0-RC1 from 0.12.2-RC4 I started getting consistent JVM crashes in all integration tests. This is with JDK 1.8.0_222-b10.

I noticed if I build rsocket-core with Dysprosium and use that instead, then spring-messaging tests are back to green. I don't know why but given Reactor Dysprosium should be ready in time for RSocket Java 1.0.0, is there any reason for RSocket Java not to aim to depend on that? If not, could the develop branch be upgraded to Dysprosium?

@robertroeser
Copy link
Member

Sounds good - can you submit a pull request to upgrade RSocket to Dysprosium?

rstoyanchev added a commit to rstoyanchev/rsocket-java that referenced this issue Jul 25, 2019
rstoyanchev added a commit to rstoyanchev/rsocket-java that referenced this issue Jul 25, 2019
Closes rsocketgh-670

Signed-off-by: Rossen Stoyanchev <rstoyanchev@pivotal.io>
rstoyanchev added a commit to rstoyanchev/rsocket-java that referenced this issue Jul 25, 2019
Closes rsocketgh-670

Signed-off-by: Rossen Stoyanchev <rstoyanchev@pivotal.io>
rstoyanchev added a commit to rstoyanchev/rsocket-java that referenced this issue Jul 25, 2019
Closes rsocketgh-670

Signed-off-by: Rossen Stoyanchev <rstoyanchev@pivotal.io>
robertroeser pushed a commit that referenced this issue Jul 25, 2019
Closes gh-670

Signed-off-by: Rossen Stoyanchev <rstoyanchev@pivotal.io>
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 a pull request may close this issue.

2 participants