avro icon indicating copy to clipboard operation
avro copied to clipboard

AVRO-1425: [Java] prioritise thread context ClassLoader over context class ClassLoader

Open sleicht opened this issue 4 months ago • 1 comments

What is the purpose of the change

Resolves AVRO-1425:

Changes the class loading strategy in ClassUtils to first try the current thread's context class loader before falling back to the context class's class loader. This improves class loading reliability in different runtime environments.

Removes the overloaded forName method that accepted a ClassLoader parameter and updates all callers to use the simplified method signature.

Verifying this change

This change is already covered by existing tests, such as TestSpecificData.

I also tested with a local SpringBoot application using Avro/Kafka and SpringBoot DevTools. Without the change I get a ClassCastException, and with the change it works as expected.

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? already working as documented

sleicht avatar Aug 19 '25 13:08 sleicht

Updated original repo which showed the issue in SpringBoot/DevTools/Avro: https://github.com/sleicht/spring-boot-devtools-with-avro

sleicht avatar Aug 19 '25 14:08 sleicht