sijie commented on issue #570: Multiple active entrylogs
URL: https://github.com/apache/bookkeeper/issues/570#issuecomment-368654064
 
 
   > It would be good if this could be implemented as a new LedgerStorage 
implementation, rather than modifying what we have already with a bunch of 
boolean flags scattered around the code. I see the patch makes modifications to 
EntryMem table for example, which is entirely unnecessary if you only write a 
single ledger to an entrylog.
   
   I think this has been discussed a while back. I would suggest no holding 
this for requesting a new ledger storage implementation. In order to make the 
community move forward with this change, I would suggest taking what Charan has 
at this moment, and review it based on that. We can consider any better 
refactoring after that. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to