temporalio / sdk-java

Temporal Java SDK
https://temporal.io
Apache License 2.0
200 stars 134 forks source link

KotlinObjectMapperFactory is not forwards compatible #2071

Open Sineaggi opened 1 month ago

Sineaggi commented 1 month ago

Expected Behavior

KotlinObjectMapperFactory works

Actual Behavior

KotlinObjectMapperFactory crashes with MethodNotFoundError

Steps to Reproduce the Problem

  1. Use temporal kotlin
  2. Upgrade jackson kotlin library
  3. Crash

Specifications

Instead of using a deprecated method, it would have been best to try using the builder first, then fall back to the deprecated KotlinModule constructor. Alternatively, we should just use the builder as we can't even pick which KotlinModule constructor we compile against.