rdblue commented on a change in pull request #31475:
URL: https://github.com/apache/spark/pull/31475#discussion_r573189665
##########
File path:
sql/catalyst/src/test/scala/org/apache/spark/sql/connector/InMemoryTable.scala
##########
@@ -51,7 +51,7 @@ class InMemoryTable(
val distribution: Distribution = Distributions.unspecified(),
val ordering: Array[SortOrder] = Array.empty)
extends Table with SupportsRead with SupportsWrite with SupportsDelete
- with SupportsMetadataColumns {
+ with SupportsMetadataColumns with TruncatableTable {
Review comment:
I think that `SupportsDelete` should implement `TruncatableTable`,
similar to how `SupportsOverwrite` implements `SupportsTruncate`. That way it
isn't necessary to for implementations to support both delete and truncate
separately.
----------------------------------------------------------------
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]
