jonahgao opened a new pull request, #3145:
URL: https://github.com/apache/kvrocks/pull/3145

   Historically, it was used for doing some checks.
   
https://github.com/apache/kvrocks/blob/2bbfe5aa9531f6e76bfd10a2cc450b9bfa0f15d9/src/storage.cc#L175-L182
   
   
   But these checks no longer exist today, so this operation should be 
unnecessary.
   
   `ListColumnFamilies()` needs to iterate through the MANIFEST, and might be 
costly for a large one.
   For example, 80MB MANIFEST, took 1.6 seconds.


-- 
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-unsubscr...@kvrocks.apache.org

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

Reply via email to