gengliangwang commented on a change in pull request #29057:
URL: https://github.com/apache/spark/pull/29057#discussion_r452600670
##########
File path: .github/workflows/master.yml
##########
@@ -1,156 +1,238 @@
name: master
on:
- push:
Review comment:
@dongjoon-hyun Sorry about using the word "smoke test" without details.
I meant we can run linter/build/maven build jobs after commits are pushed, just
like what it is now.
> please note that this PR can cause a significant delay if there is no
enough resource given from ASF foundation.
That's my concern here. If we run all the tests on push, then there might be
too many queued jobs.
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
