platoneko opened a new pull request, #38808:
URL: https://github.com/apache/doris/pull/38808

   ## Proposed changes
   
   In path v1, the segment file path is 
"data/${shard_id}/${tablet_id}/${rowset_id}/${seg_id}.dat", which means that 
segment files of different rowsets can have the same file name (e.g. "0.dat"). 
However, the initial file cache key is based on the file name, which will lead 
to cache key collisions. Additionally, to maintain the generality of 
`CachedRemoteFileReader`, we should not assume that all files accessed by 
`CachedRemoteFileReader` have globally unique file names. Therefore, it is 
necessary to use the full path as the cache key.


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to