apache/pinot
Vedi su GitHubAdd sanity check to the output of Realtime Provisioning Tool
Open
#8339 aperta il 11 mar 2022
good first issue
Metriche repository
- Star
- (4937 star)
- Metriche merge PR
- (Merge medio 6g 7h) (186 PR mergiate in 30 g)
Descrizione
Currently RTP tool generates a mutable segments based on the given schema characteristics (or a given sample segment) and then linearly extrapolates the required memory, segment size, etc for different combinations of the input parameters. We've recently seen people have used 5GB segments because of the tool's suggestion. It's fine if the tool outputs 5GB because of the chosen input parameters, but it should give some warnings about potential problematic values. Here are some suggestions:
- Change the output data structure to include an array of warnings
- If a suggested segment size is > 500MB, add a warning message that the desired segment size should be < 500MB.
- If number of segments is greater than a threshold (maybe 5K?), then the tools add a warning for that.
- Sometimes tools outputs NA if the memory goes above the provided availableHostMemory parameter. Instead of NA, we should output the actual value, but add a warning that memory for some combinations goes above available host memory