HyukjinKwon commented on a change in pull request #25838: [SPARK-29158][SQL]
Expose SerializableConfiguration for DataSource V2 developers
URL: https://github.com/apache/spark/pull/25838#discussion_r325966883
##########
File path:
core/src/main/scala/org/apache/spark/util/SerializableConfiguration.scala
##########
@@ -20,7 +20,13 @@ import java.io.{ObjectInputStream, ObjectOutputStream}
import org.apache.hadoop.conf.Configuration
-private[spark]
+import org.apache.spark.annotation.DeveloperApi
+
+/**
+ * Helper wrapper to serialize a Hadoop configuration. Intended for use when
implementing
+ * DataSourceV2 readers & writers which depend on the Hadoop configuration
from the driver node.
+ */
+@DeveloperApi
Review comment:
Shall we add `@Unstable` to stay safe, and can you check if this is callable
in Java side as well?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
