Vladimir Sitnikov created CALCITE-7852:
------------------------------------------
Summary: Geode schema fails with NullPointerException when
pdxSerializablePackagePath is not set
Key: CALCITE-7852
URL: https://issues.apache.org/jira/browse/CALCITE-7852
Project: Calcite
Issue Type: Bug
Components: geode-adapter
Reporter: Vladimir Sitnikov
Fix For: 1.43.0
A Geode schema whose model does not set the {{pdxSerializablePackagePath}}
operand cannot be created when Calcite has to create a new Geode client cache.
Since CALCITE-7787 the operand is rejected unless the system property
{{calcite.geode.pdxSerializablePackagePath.allowed}} is {{true}}, so leaving it
out is the only configuration that works by default, and that configuration
fails.
Both {{GeodeSchemaFactory}} and {{GeodeSimpleSchemaFactory}} fail the same way.
Reproduced on main at 8f27e24a02 (1.43.0-SNAPSHOT, geode-core 1.15.1) by
creating the schema from the operand that {{GeodeSchemaFactoryTest.operand()}}
builds, minus {{pdxSerializablePackagePath}}, after
{{GeodeUtils.closeClientCache()}}:
{code}
java.lang.NullPointerException: Cannot invoke "String.split(String)" because
"value" is null
at
org.apache.geode.pdx.internal.AutoSerializableManager.processInitParams(AutoSerializableManager.java:2370)
at
org.apache.geode.pdx.internal.AutoSerializableManager.reconfigure(AutoSerializableManager.java:2362)
at
org.apache.geode.pdx.internal.AutoSerializableManager.create(AutoSerializableManager.java:148)
at
org.apache.geode.pdx.ReflectionBasedAutoSerializer.<init>(ReflectionBasedAutoSerializer.java:212)
at
org.apache.geode.pdx.ReflectionBasedAutoSerializer.<init>(ReflectionBasedAutoSerializer.java:166)
at
org.apache.calcite.adapter.geode.util.GeodeUtils.createClientCache(GeodeUtils.java:126)
at
org.apache.calcite.adapter.geode.rel.GeodeSchemaFactory.create(GeodeSchemaFactory.java:71)
{code}
Expected: {{create}} returns a {{GeodeSchema}} (or a {{GeodeSimpleSchema}})
whose client cache has a {{ReflectionBasedAutoSerializer}} with no class
pattern, as {{new ReflectionBasedAutoSerializer()}} creates.
{{GeodeUtils.checkPdxPackagePathOperand}} documents a null operand as "not set"
and accepts it, so the factories are meant to work without it.
Cause: {{GeodeUtils.createClientCache}} passes the null operand to {{new
ReflectionBasedAutoSerializer(String...)}}, which reads it as a class pattern.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
