Merge branch 'develop' into integration #1673
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
) This reverts commit fa731f4. Co-authored-by: Amol Nayak <[email protected]>
…db#1610) Today scaling filoDB horizontally involves re-calculation of memory settings in a manual way involving tribal knowledge. This PR aims to automate it and make it simple math. It is backward compatible and behind feature flag. * Min-num-nodes moved from dataset into server config. * New server config will drive automatic memory calculation * Each dataset requires another configuration that determines what fraction of resources each dataset gets.
* filodb(core) add debugging info for empty histogram. Some queries occasionally hit exceptions because of empty histogram. However, the same exception could not be reproduced later. The hunch is that the bug is caused by a race condition. So, adding additional debugging log to print out the chunk id chunk info and the memory dump. --------- Co-authored-by: Yu Zhang <[email protected]> Co-authored-by: alextheimer <[email protected]> Co-authored-by: sandeep6189 <[email protected]>
Co-authored-by: Yu Zhang <[email protected]>
…1613)" (filodb#1623) This reverts commit 90303aa.
Co-authored-by: Kier Petrov <[email protected]>
…ata scans (filodb#1628) Double.isNan involves conversion to boxed java Double Local heap profiling showed that this is a significant allocation. Conversion to the static java.lang.Double.isNan removes these.
skip busting the problematic entry. Co-authored-by: Yu Zhang <[email protected]>
Some queries occasionally hit exceptions because of empty histogram. However, the same exception could not be reproduced later. The hunch is that the bug is caused by a race condition. So, adding additional debugging log to print out the chunk id chunk info and the memory dump. Co-authored-by: Yu Zhang <[email protected]>
There are two configs for num-nodes used by clustering-v2 and automatic memory alloc code. Consolidating them.
Default alloc configs should sum to 100.
…n without the label (filodb#1639)
filodb#1629) * fix(core) fix the unless operator for aggregators. For regex shard key we need to aggregate across all nodes. InProcessPlanDispatcher is needed. --------- Co-authored-by: Yu Zhang <[email protected]>
feat(query): Cardinality V2 API Query Plan changes
convert unary expressions through binary expressions. Co-authored-by: Yu Zhang <[email protected]>
) There was a bug in calculating size of SRV. Earlier, for efficiency purposes, we were calculating the size of the containers associated with the SRV. But actually, the container can home multiple SRVs. So the calculated size for several SRVs at a time can end up wrong with addition of cumulative counts. The fix for now is to calculate the size by going through the records. It introduces a small inefficiency here, but submitting this PR for now since other ways to calculate this were more invasive and risk regression. We can have an optimization of this if really needed later. I have also reduced the number of calls to this method from two to one. The unit tests didn't catch this since earlier since they played with one SRV only. I have now added a unit test that adds multiple SRVs. It failed with earlier code.
… enum and prefix regex filters (filodb#1641) Production profiling is showing that Lucene Regex automata is creating a hotspot in method and allocation profiling. This PR optimizes two kinds of queries * Regex with enumerated values are converted to TermInSetQuery * Regex with prefix is converted to PrefixQuery It also wraps Lucene queries in ConstantScoreQuery to prevent any scoring that may be happening. Observed 2.2x performance improvement in JMH benchmark for specific enum regex query 1.5x performance improvement in JMH benchmark for specific prefix regex query
…c calls (filodb#1651) * Adding UserDatasets for remote calls * Updating UT
Adding CPU Nanos Time consumed for Lucene index lookups in query stats, especially since we are seeing this in frequently as a hotspot in CPU profile information
…anner for TenantIngestionMetering changes
…s for cardinality calculation time (filodb#1666) * Adding config support for DS Card flushCount and perf logs for cardinality calculation time
sandeep6189
approved these changes
Sep 19, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request checklist
Current behavior : (link exiting issues here : https://help.github.com/articles/basic-writing-and-formatting-syntax/#referencing-issues-and-pull-requests)
New behavior :
BREAKING CHANGES
If this PR contains a breaking change, please describe the impact and migration
path for existing applications.
If not please remove this section.
Breaking changes may include:
Other information: