cloud-fan commented on a change in pull request #31271:
URL: https://github.com/apache/spark/pull/31271#discussion_r561621419
##########
File path:
core/src/main/scala/org/apache/spark/scheduler/ExecutorDecommissionInfo.scala
##########
@@ -31,6 +31,8 @@ case class ExecutorDecommissionInfo(message: String,
workerHost: Option[String]
* State related to decommissioning that is kept by the TaskSchedulerImpl.
This state is derived
* from the info message above but it is kept distinct to allow the state to
evolve independently
* from the message.
+ *
+ * @since 3.1.0
Review comment:
@holdenk @Ngone51 do we mean to expose this class to end-users?
##########
File path:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
##########
@@ -1268,7 +1268,7 @@ private[hive] object HiveClientImpl extends Logging {
}
}
-case object HiveVoidType extends DataType {
+private[hive] case object HiveVoidType extends DataType {
Review comment:
hmmm, I thought `org.apache.spark.sql.hive` is already a private
package, no? cc @viirya @dongjoon-hyun
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
