apache / kyuubi

Apache Kyuubi is a distributed and multi-tenant gateway to provide serverless SQL on data warehouses and lakehouses.
https://kyuubi.apache.org/
Apache License 2.0
2.11k stars 916 forks source link

[Improvement] disable periodic GC interval if not necessary #6704

Closed taylor12805 closed 1 month ago

taylor12805 commented 2 months ago

Code of Conduct

Search before asking

What would you like to be improved?

The feature of periodically GC: https://github.com/apache/kyuubi/issues/4352 is very necessary in some scenario, but in our case, System.gc() seems like not collect any week reference and might cause a long period of Stop-the-World. So it might be necessary to not trigger periodically GC if configuration kyuubi.server.periodicGC.interval is set to 0

How should we improve?

if configuration kyuubi.server.periodicGC.interval is set to 0, not trigger periodically GC

Are you willing to submit PR?

github-actions[bot] commented 2 months ago

Hello @taylor12805, Thanks for finding the time to report the issue! We really appreciate the community's efforts to improve Apache Kyuubi.