dongjoon-hyun edited a comment on pull request #31303: URL: https://github.com/apache/spark/pull/31303#issuecomment-766457914
Actually, I understand the requirement and the intention because I also reported SPARK-33822 (TPCDS Q5 fails if spark.sql.adaptive.enabled=true). > I remember we faced a case that TPSDS was broken whereas the Spark tests pass. @wangyum Could you provide a result in your repository which is requested by @HyukjinKwon ? It depends on the characteristic of the job. > Is the perf inconsistency big? It will give you a rough numbers to check. BTW, on top of the above issues, this is not safe in terms of security. Recently, Apache Infra team introduced strict security enforcements by banning the runnable source of GitHub Action. We have no available option when the 3rd party repositories are compromised . Even the docker image repo, we should migrate `ApacheSparkGitHubActionImage` to Spark's official branch. For the two repositories in this PR, the migration is possible? ---------------------------------------------------------------- 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
