snuyanzin commented on code in PR #28129:
URL: https://github.com/apache/flink/pull/28129#discussion_r3217992493
##########
flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/planner/codegen/CodeGeneratorContext.scala:
##########
@@ -145,6 +145,11 @@ class CodeGeneratorContext(
private val localRefScopes =
mutable.ArrayBuffer(mutable.LinkedHashMap.empty[Int, GeneratedExpression])
+ // cache of pre-parsed JSON variables keyed by input result term, so that
multiple
+ // JSON function calls on the same input share a single parse
+ private val reusableParsedJsonExprs: mutable.Map[String, String] =
Review Comment:
I replaced with using existing `addReusableInputUnboxingExprs` since it
already allows to put/get via map
--
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]
