Skip to content

Spring Boot Upgrade#226

Closed
pmonington wants to merge 53 commits intodevelopfrom
pmonington/openrewrite/upgrade-spring
Closed

Spring Boot Upgrade#226
pmonington wants to merge 53 commits intodevelopfrom
pmonington/openrewrite/upgrade-spring

Conversation

@pmonington
Copy link
Copy Markdown

@pmonington pmonington commented Apr 10, 2026

PR Details

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

  • Defect fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that cause existing functionality to change)

Checklist:

  • I have added any new packages to the sonar-scanner.properties file
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
    ODE Contributing Guide
  • I have added tests to cover my changes.
  • All new and existing tests passed.

pmonington added 30 commits March 12, 2026 15:37
@pmonington
tool: Add Rewrite Maven plugin for Spring Boot 4 upgrade
0f1a24e
@pmonington
tool: run open rewrite recipe on jpo-ode-svcs for Spring upgrade
7fb296a
@pmonington
tool: run open rewrite recipe on jpo-ode-core for Spring upgrade
676526b
@pmonington
tool: run open rewrite recipe on jpo-ode-plugins for Spring upgrade
b685fc2
@pmonington
tool: run open rewrite recipe on jpo-ode-common for Spring upgrade
3a04dac
@pmonington
tool: run open rewrite recipe on jpo-ode pom file upgrading spring bo…
1f48fe1 
…ot to 4
@pmonington
fix: refactor XmlUtils and JsonUtils to use builder APIs for Jackson …
1173e1e 
…Mapper initialization
@pmonington
refactor: update JsonUtils and tests to use Assertions and improve ty…
2ac15cd 
…pe handling
@pmonington
refactor: remove redundant "throws IOException" from EncodingsDeseria…
a92f4d4 
…lizer deserialize method
@pmonington
fix: correct iterator method in TravelerMessageFromHumanToAsnConverter
4deb8c9
@pmonington
chore: remove unnecessary longValue() cast
44b65d4
@pmonington
fix: set default value for asBoolean conversions in builders to avoid…
ff37447 
… Json serialization error
@pmonington
fix: set default values for asLong conversions instead of throwing ex…
083b76c 
…ceptions when encountering "unavailable"
@pmonington
refactor: update MetricsConfig imports from spring boot upgrade
aa1d47c
@pmonington
refactor: change exception thrown by GenericReceiver to throw custom …
9b10eb8 
…UnsupportedMessageTypeException
@pmonington
chore: deprecate unused classes and disable related unit tests
0146bbd
@pmonington
fix: update RestTemplate configuration to use JacksonJsonHttpMessageC…
3762e39 
…onverter and JsonMapper
@pmonington
test: alphabetize the expected output of tests to match the default a…
7c0d1c9 
…lphabetization of Jackson 3
@pmonington
chore: remove obsolete kafka_2.11 dependency from pom.xml
c972fdc
@pmonington
fix: adjust object mappers to comply with jackson 3
2853d3a
@pmonington
fix: update EmbeddedKafkaHolder to use EmbeddedKafkaKraftBroker inste…
65a0cb7 
…ad of EmbeddedKafkaZKBroker for transition to kraft instead of zookeeper
@pmonington
chore: deprecate JsonUtils and related methods in OdeObject, suggest …
3aed42a 
…using Spring context mappers
@pmonington
docs: adjust commentation to add reason for disabling default alphabe…
6f591d7 
…tical sorting
@pmonington
test: add setup method to initialize topics in RawEncodedBSMJsonRoute…
763175d 
…rTest and refactor EmbeddedKafkaHolder
@pmonington
Revert "test: alphabetize the expected output of tests to match the d…
bd9e4d2 
…efault alphabetization of Jackson 3"

This reverts commit 7c0d1c9.
@pmonington
test: revert alphabetization of expected toJson output
21c6421
@pmonington
test: refactor RawEncodedBSMJsonRouterTest to use @embeddedkafka and …
2b5c3b9 
…@KafkaListener, remove EmbeddedKafkaHolder usage
@pmonington
test: replace EmbeddedKafkaHolder with @embeddedkafka and @KafkaListe…
447793b 
…ner in all tests, refactor tests to use CountDownLatch for async validation
@pmonington
test: refactor RsmReceiverTest to use @embeddedkafka and @KafkaListen…
1281c42 
…er, remove EmbeddedKafkaHolder usage
@pmonington
test: mark testSdsmSerializationDeserialization temporarily as @disabled
870079f 
 in OdeMessageFrameDataTest to run kafka tests
pmonington added 21 commits March 26, 2026 10:31
@pmonington
refactor: add legacy ObjectMapper bean in SerializationConfig
4ef688b
@pmonington
test: refactor SrmReceiverTest to use @embeddedkafka and @KafkaListen…
e17bb6c 
…er, remove EmbeddedKafkaHolder usage
@pmonington
test: refactor SsmReceiverTest to use @embeddedkafka and @KafkaListen…
5e2101a 
…er, remove EmbeddedKafkaHolder usage
@pmonington
test: refactor RtcmReceiverTest to use @embeddedkafka and @KafkaListe…
11694f9 
…ner, replace EmbeddedKafkaHolder with CountDownLatch for async validation
@pmonington
test: refactor SdsmReceiverTest to use @embeddedkafka and @KafkaListe…
6c32458 
…ner, replace EmbeddedKafkaHolder with CountDownLatch for async validation
@pmonington
chore: remove unused import
de35fd1
@pmonington
test: refactor PsmReceiverTest to use @embeddedkafka and @KafkaListen…
90808bf 
…er, replace EmbeddedKafkaHolder with CountDownLatch for async validation
@pmonington
test: refactor SpatReceiverTest to use @embeddedkafka and @KafkaListe…
06aaed8 
…ner, replace EmbeddedKafkaHolder with CountDownLatch for async validation
@pmonington
test: remove EmbeddedKafkaHolder usage in SpatReceiverTest
34c7bd3
@pmonington
test: refactor MapReceiverTest to use @embeddedkafka and @KafkaListener
63b5986
@pmonington
test: refactor RawEncodedSRMJsonRouterTest to use @embeddedkafka and …
cb65fab 
…@KafkaListener, replace EmbeddedKafkaHolder with CountDownLatch for async validation
@pmonington
test: remove unused imports in RawEncodedSRMJsonRouterTest
14b3183
@pmonington
test: refactor RawEncodedSSMJsonRouterTest to use @embeddedkafka and …
c0682e0 
…@KafkaListener
@pmonington
test: refactor RawEncodedTIMJsonRouterTest to use @embeddedkafka and …
ff56874 
…@KafkaListener
@pmonington
test: disable broken test Asn1DecodedDataRouterTIMDataFlow with Disab…
08753e5 
…led annotation and associated ticket number
@pmonington
refactor: add legacy Jackson ObjectMapper implementations to JsonUtil…
f915389 
…s and support for deserialization use cases
@pmonington
refactor: add legacy XML mapper bean to SerializationConfig
9a25a0e
@pmonington
refactor: add legacyToJson method to JsonUtils for object-to-JSON con…
be8ce7b 
…version with optional null inclusion
@pmonington
refactor: update OdeMessageFrameDataCreatorHelper to use legacy Jacks…
f9ef402 
…on XmlMapper
@pmonington
build: add jackson-databind dependency to pom.xml for legacy Jackson …
92037f0 
…support
@pmonington
refactor: use legacy XmlMapper bean to simpleLegacyXmlMapper for Asn1…
a841f3a 
…DecodedDataRouter
@mcook42 mcook42 marked this pull request as draft April 10, 2026 17:43
@pmonington pmonington closed this Apr 23, 2026
@pmonington
Copy link
Copy Markdown
Author

pmonington commented Apr 23, 2026

New PR was created to upgrade spring without the optional Jackson 3 upgrade: #227.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@pmonington