viirya commented on code in PR #613:
URL: https://github.com/apache/datafusion-comet/pull/613#discussion_r1672515755
##########
spark/src/test/scala/org/apache/spark/sql/CometTPCDSQuerySuite.scala:
##########
@@ -112,7 +108,9 @@ class CometTPCDSQuerySuite
"q69",
"q70",
"q71",
- "q72",
+ // TODO: unknown failure (seems memory usage over Github action
runner) in CI with q72 in
+ // https://github.com/apache/datafusion-comet/pull/613.
+ // "q72",
Review Comment:
> I do think we should still test with the original q72 as a separate
exercise though, because if Spark can run it then Comet should be able to as
well (with the same memory configuration).
Yea. As I mentioned earlier, I will investigate q72 further to see why it
requires extra memory in Comet. Just disable it to unblock this PR.
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
