aokolnychyi commented on code in PR #51002: URL: https://github.com/apache/spark/pull/51002#discussion_r2105478330
########## sql/api/src/main/scala/org/apache/spark/sql/types/Metadata.scala: ########## @@ -40,11 +40,13 @@ import org.apache.spark.util.ArrayImplicits._ * @since 1.3.0 */ @Stable -sealed class Metadata private[types] (private[types] val map: Map[String, Any]) +sealed class Metadata private[types] ( + private[types] val map: Map[String, Any], + @transient private[types] val runtimeMap: Map[String, Any]) Review Comment: I am adding a runtime-only map of configs that is not serialized or exposed to the user. It will allow me to store alternative in-memory representations for certain configs. In particular, it would allow me to store SQL as well as the expression itself for default values. -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
