dongjoon-hyun opened a new pull request, #56953:
URL: https://github.com/apache/spark/pull/56953

   ### What changes were proposed in this pull request?
   
   This PR proposes to skip scheduling the periodic GC task in 
`ContextCleaner.start()` when `spark.cleaner.periodicGC.interval` is 
non-positive, and documents that such a value disables the periodic GC.
   
   ### Why are the changes needed?
   
   Previously, `spark.cleaner.periodicGC.interval=0s` failed `SparkContext` 
creation because the value was passed directly to `scheduleAtFixedRate`, which 
throws `IllegalArgumentException` when `period <= 0`. This change makes a 
non-positive value a way to disable the periodic `System.gc()`.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No because this is a new feature. Previously, it fails with 
`IllegalArgumentException` at the start-up.
   
   ### How was this patch tested?
   
   Pass the CIs with the newly added test case.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Fable 5


-- 
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