imply-cheddar commented on code in PR #13947:
URL: https://github.com/apache/druid/pull/13947#discussion_r1141616693
##########
processing/src/main/java/org/apache/druid/math/expr/ExprEval.java:
##########
@@ -515,6 +515,9 @@ public static ExprEval ofType(@Nullable ExpressionType
type, @Nullable Object va
if (value instanceof List) {
return bestEffortOf(value);
}
+ if (value instanceof byte[]) {
+ return new StringExprEval(StringUtils.encodeBase64String((byte[])
value));
+ }
Review Comment:
This has me wondering, what if an expression actually wants the `byte[]` how
would it be defined so that if one expression returns a `byte[]` and the next
one wants to use it, then it will just be passed through without being base64
encoded in between?
--
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]
