andygrove commented on code in PR #562:
URL: https://github.com/apache/datafusion-comet/pull/562#discussion_r1637003741
##########
spark/src/test/scala/org/apache/comet/CometExpressionCoverageSuite.scala:
##########
@@ -80,62 +80,102 @@ class CometExpressionCoverageSuite extends CometTestBase
with AdaptiveSparkPlanH
// This can be simplified once comet supports MemoryScan, now Comet triggers
from the FileScan
// If MemoryScan supported we can just run Spark examples as is
val manualTests: Map[String, (String, String)] = Map(
- "!" -> ("select true a", "select ! true from tbl"),
Review Comment:
compiler warning: Adapting argument list by creating a 2-tuple: this may not
be what you want.
--
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]
