Anurag Rai
Anurag Rai
+1 , I think a data-only CRC instead of completely skipping CRC check might be better here. There are still more corner cases with data only CRCs as well. eg....
Instead of upstreaming the skip CRC check - I would propose replacing it with a data only CRC. Added details in [this issue. ](https://github.com/apache/pinot/issues/17262) , which can keep it both...
For this, >We should set the creationTime = max (creationTime of all merging segments) + 1 to avoid this and fully replace all records from merging segments with new merged...
@Jackie-Jiang PTAL as well.
> We need to discuss when should we use data CRC instead of index CRC, and what is the side effect. When using data CRC, index only change happening in...
> > > We need to discuss when should we use data CRC instead of index CRC, and what is the side effect. When using data CRC, index only change...
CC @rohityadav1993 @Jackie-Jiang