clintropolis commented on a change in pull request #11893:
URL: https://github.com/apache/druid/pull/11893#discussion_r745275959
##########
File path:
processing/src/main/java/org/apache/druid/segment/serde/ComplexMetrics.java
##########
@@ -69,4 +69,16 @@ public static void registerSerde(String type,
ComplexMetricSerde serde)
}
});
}
+
+ /**
+ * Unregister a serde name -> ComplexMetricSerde mapping.
+ *
+ * If the spedified serde key string is not in use, does nothing.
Review comment:
```suggestion
* If the specified serde key string is not in use, does nothing.
```
##########
File path:
processing/src/main/java/org/apache/druid/segment/serde/ComplexColumnPartSerde.java
##########
@@ -78,12 +78,14 @@ public Serializer getSerializer()
public Deserializer getDeserializer()
{
return (buffer, builder, columnConfig) -> {
+ // we don't currently know if complex column can have nulls (or can be
multi-valued, but not making that change
+ // since it isn't supported anywhere in the query engines)
+ // longer term this needs to be captured by making the serde provide
this information, and then this should
+ // no longer be set to true but rather the actual values
+ builder.setHasNulls(ColumnCapabilities.Capable.TRUE);
+ builder.setComplexTypeName(typeName);
Review comment:
:+1:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
