Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate the lite-init-stats configuration #60208

Open
hawkingrei opened this issue Mar 21, 2025 · 0 comments
Open

Deprecate the lite-init-stats configuration #60208

hawkingrei opened this issue Mar 21, 2025 · 0 comments
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@hawkingrei
Copy link
Member

hawkingrei commented Mar 21, 2025

Enhancement

TL;DR

The lite-init-stats is good enough, and there is no need for non-lite-init-stats. 

The lite-init-stats configuration was introduced to provide a lightweight way to initialize statistics for query optimization. However, with the evolution of our system and the improvements in the sync load mechanism, the necessity of lite-init-stats has diminished.

Sufficient Completeness of Sync Load

  • current sync load mechanism has been significantly enhanced and is now robust enough to handle the initialization of statistics without the need for a lightweight alternative. It ensures that statistics are loaded efficiently and accurately, even under various workloads and scenarios.

  • The previous concerns about sync load potentially causing timeouts have been addressed. The system now has better timeout handling and resource management, making it a reliable method for initializing statistics.

Memory and Complexity Considerations

  • non lite-init-stats will use more memory and load unnecessary data into memory.
  • Because we need to manage stats status into memory to control whether it need to sync load. so it will add more complexity.

Documentation and User Guidance

  • The documentation has already been updated to remove recommendations for disabling lite-init-stats. This indicates a shift in our guidance towards using the more comprehensive and reliable lite initialization method.

  • Users are now encouraged to rely on the default statistics initialization mechanism, which aligns with the improvements in sync load and the overall system performance.

@hawkingrei hawkingrei added the type/enhancement The issue or PR belongs to an enhancement. label Mar 21, 2025
@hawkingrei hawkingrei changed the title Deprecate the lite-init-stats configuration Deprecate the lite-init-stats configuration Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

No branches or pull requests

1 participant