[KYUUBI #6704] Disable periodic gc if set interval to 0

[taylor12805]

🔍 Description

Issue References 🔗

This pull request fixes #6704

Describe Your Solution 🔧

if periodic gc is set to 0, there is no need to perform an explicit gc.

Types of changes 🔖

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Behavior Without This Pull Request ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests

---

Checklist 📝

• This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (2d64255) to head (a52ddda).
Report is 5 commits behind head on master.

Files with missing lines	Patch %	Lines
...a/org/apache/kyuubi/server/PeriodicGCService.scala	0.00%	8 Missing ⚠️
...in/scala/org/apache/kyuubi/config/KyuubiConf.scala	0.00%	1 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff           @@
##           master   #6725   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         684     684           
  Lines       42282   42283    +1     
  Branches     5767    5768    +1     
======================================
- Misses      42282   42283    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pan3793]

behavior change looks good to me, would you mind updating the doc of kyuubi.periodicGC.interval to mention that?

[bowenliang123]

@taylor12805 Please update the document for kyuubi.server.periodicGC.interval in KyuubiConf class, and then run dev/gen/gen_all_config_docs.sh script to update the markdown docs. Thanks for your contribution.

[taylor12805]

behavior change looks good to me, would you mind updating the doc of kyuubi.periodicGC.interval to mention that?

@taylor12805 Please update the document for kyuubi.server.periodicGC.interval in KyuubiConf class, and then run dev/gen/gen_all_config_docs.sh script to update the markdown docs. Thanks for your contribution.

sure, i have updated the document

[pan3793]

@taylor12805 you need to modify the code and regenerate the md

kyuubi/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala

Line 3112 in f8431da

.doc("How often to trigger a garbage collection.")

[taylor12805]

@taylor12805 you need to modify the code and regenerate the md

kyuubi/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala

Line 3112 in f8431da

.doc("How often to trigger a garbage collection.")

yeah, sure, i have figured that out

[pan3793]

Thanks, merged to master for 1.10