decster opened a new pull request #3762:
URL: https://github.com/apache/incubator-doris/pull/3762


   After adding meta, BE now can create MemTablet, and put it into 
TabletManager, but MemTablet is not compatible with Tablet, a lot of code may 
break. This CL contains the following changes:
   
   When TabletManager::get_tablet is called, only return Tablet, if the 
underlying tablet is MemTablet, return an error, this keeps the initial code 
changes small.
   
   For methods/functionalities Tablet&MemTablet both have, refactor/extract 
them to BaseTablet, and change the usage to TabletManager::get_base_tablet, and 
it will gradually remove errors introduced by step1, this is a long going 
process.


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

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to