[GitHub] [kvrocks] git-hulk commented on pull request #1574: Style: Change Storage::SetReadOptions to DefaultScanOptions

["git-hulk (via GitHub)" <gi...@apache.org>]

git-hulk commented on PR #1574:
URL: https://github.com/apache/kvrocks/pull/1574#issuecomment-1629154905

   > @git-hulk By the way, when would kvrocks use `fill_cache = true`? Seems that only when point get, 
   
   Yes, only point lookup would enable the fill_cache option. 
   
   > we would have that, and read_ahead / fill_cache is usally disabled for performance?
   
   Sorry, I didn't get this point.  Do you mean that should also enable the fill_cache for the seek operation?


-- 
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: issues-unsubscribe@kvrocks.apache.org

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