You are viewing a plain text version of this content. The canonical link for it is here.

Posted to pr@cassandra.apache.org by GitBox <gi...@apache.org> on 2020/09/22 16:42:50 UTC

[GitHub] [cassandra] maedhroz commented on pull request #750: CASSANDRA-15164 log unusual EstimatedHistogram serializations and deserializations

maedhroz commented on pull request #750:
URL: https://github.com/apache/cassandra/pull/750#issuecomment-696841583


   > when that metric in the sstable is overflowed it will cause errors throughout the rest of the codebase that isn't caught
   
   Yeah, there are a couple other places that might still explode, like GROUP BY queries and range reads if we allow an overflowed histogram. Short of dynamically resizing the number of buckets, I'd propose that for now we sanitize the two estimate histograms in `StatsMetadataSerializer`, so we can at least differentiate the WARN messages in the log.
   
   CC @blerer 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org