You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/02/22 00:26:18 UTC

[GitHub] [spark] HyukjinKwon edited a comment on pull request #31603: [SPARK-34029][SQL][TESTS] Add OrcEncryptionSuite and FakeKeyProvider

HyukjinKwon edited a comment on pull request #31603:
URL: https://github.com/apache/spark/pull/31603#issuecomment-782957864


   I don't have a strong opinion on this. This added 30min GA job to test one case and I would prefer to avoid adding this given that we face GA resource issues globally going on.
   
   Do we know the reason why it needs to run the test in a separate forked JVM to pass? I know its tricky to investigate such case but it might be worthwhile taking a look to avoid adding another GA job.
   
   Otherwise I would name the tag to something like DedicatedJVM and add some more test when we face touch issue next time.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org