mridulm commented on code in PR #36473:
URL: https://github.com/apache/spark/pull/36473#discussion_r867044023
##########
core/src/test/scala/org/apache/spark/storage/DiskBlockManagerSuite.scala:
##########
@@ -141,28 +142,48 @@ class DiskBlockManagerSuite extends SparkFunSuite with
BeforeAndAfterEach with B
assert(attemptId.equals("1"))
}
+ // Use jnr to get and override the current process umask.
+ // Expects the input mask to be an octal number
+ private def getAndSetUmask(posix: POSIX, mask: String): String = {
+ val prev = posix.umask(BigInt(mask, 8).toInt)
Review Comment:
We dont currently get/set process umask. I did see use of jna in hadoop, but
that is shaded now.
Any library which allows us to set process umask would do actually, jnr
simply seemed to be more easy to use across platforms (this is for tests
anyway).
--
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: reviews-unsubscr...@spark.apache.org
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
