viirya commented on code in PR #50631:
URL: https://github.com/apache/spark/pull/50631#discussion_r2064179926
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/constraints.scala:
##########
@@ -92,6 +98,11 @@ trait TableConstraint {
)
}
}
+
+ override def nullable: Boolean = true
+
+ override def dataType: DataType =
+ throw new
SparkUnsupportedOperationException("CONSTRAINT_DOES_NOT_HAVE_DATA_TYPE")
Review Comment:
I'm not sure. Is there a case you will use nullable without dataType? Should
it be unsupported too by default for nullable?
Not a big deal, though. Just wondering the reason.
--
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
