parthchandra commented on issue #1964:
URL: 
https://github.com/apache/datafusion-comet/issues/1964#issuecomment-3024824685

   > One idea I have is to only run these workflows if the PR title contains 
`[iceberg]`.
   
   I like this idea. Run either the Spark SQL tests or the Iceberg tests 
depending on whether the PR is tagged `[iceberg]`. @kazuyukitanimura do you 
think this would mitigate any iceberg issues we have.
   
   Separately, since we have contributed the Comet integration to Iceberg, I 
feel it might be a good idea to contribute a CI to run Iceberg/Comet/Spark 
tests to the Iceberg project as well. (The issues we have seen could have been 
caught on the Iceberg side, particularly since we cannot test any changes 
unless both Comet and Iceberg are updated).


-- 
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

Reply via email to