uros-db commented on code in PR #48459:
URL: https://github.com/apache/spark/pull/48459#discussion_r1800782417
##########
sql/api/src/main/scala/org/apache/spark/sql/internal/types/AbstractStringType.scala:
##########
@@ -21,25 +21,35 @@ import org.apache.spark.sql.internal.SqlApiConf
import org.apache.spark.sql.types.{AbstractDataType, DataType, StringType}
/**
- * AbstractStringType is an abstract class for StringType with collation
support. As every type of
- * collation can support trim specifier this class is parametrized with it.
+ * AbstractStringType is an abstract class for StringType with collation
support.
*/
-abstract class AbstractStringType(private[sql] val supportsTrimCollation:
Boolean = false)
+abstract class AbstractStringType(supportsTrimCollation: Boolean = false)
Review Comment:
can we keep it private to SQL in all of them? no need to expose stuff unless
there's a need to expose stuff
--
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]
