Skip to content

Fix for Query Optimization #696

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

Merged
merged 6 commits into from
Mar 12, 2024

Conversation

nikhilsinhaparseable
Copy link
Contributor

modified function that checks if query has starttime before the 1st manifest lower bound time

to ensure server uses manifest for count(*) where starttime is greater than manifest creation date

Fixes #695

Description


This PR has:

  • been tested to ensure log ingestion and log query works.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

…anifest lower bound time

to ensure server uses manifest for count(*) where starttime is greater than manifest creation date
@nitisht nitisht merged commit a04dd4f into parseablehq:main Mar 12, 2024
nikhilsinhaparseable added a commit to nikhilsinhaparseable/parseable that referenced this pull request Apr 20, 2024
modified function that checks if query has starttime before the 1st manifest lower bound time
to ensure server uses manifest for count(*) where starttime is greater than manifest creation date

fix for ingestion with time partition
fix for query with time partition
fix for query optimization
fixed get_first_event call for time partition
random number generation logic changed while creating parquet file name
@nikhilsinhaparseable nikhilsinhaparseable deleted the query-optimization branch July 12, 2025 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Server is not using the catalog (manifest.json) file properly
2 participants