pan3793 commented on code in PR #6645:
URL: https://github.com/apache/kyuubi/pull/6645#discussion_r1741981473


##########
kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala:
##########
@@ -3121,6 +3121,14 @@ object KyuubiConf {
       .timeConf
       .createWithDefaultString("P14D")
 
+  val SERVER_TEMP_FILE_EXPIRE_MAX_COUNT: ConfigEntry[Long] =
+    buildConf("kyuubi.server.tempFile.maxCount")
+      .doc("The upper threshold size of server-side temporary file paths to 
cleanup")
+      .version("1.10.0")
+      .serverOnly
+      .longConf
+      .createWithDefault(10000)

Review Comment:
   14d might be long enough, 10000 is relatively small, how about disabling the 
size-based eviction strategy by default?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to