caiconghui commented on pull request #8574: URL: https://github.com/apache/incubator-doris/pull/8574#issuecomment-1075185517
> > > 1. Does the command line of meta_tool also need to remove schema_hash? > > > 2. Do all the http_action of be has schema_hash should remove schema_hash? If so, you should change the url of fe in `show tablet` > > > 3. if the above is yes, document is also need to change > > > > > > @yangzhg now, for upgrade compatibility , it is not the time to modify meta_tool, and url of show tablet command, it just optimize the logic in be to find tablet and drop tablet now > > I think it's understandable to bring some compatibility problems when a big version is released, and there are not many users of meta_tool and url, so it's better to modify them together I find that I cannot refactor all schema hash code because the schema_hash dir now already in data path, so in order to be less error prone, I just simplify the get tablet and drop tablet logic in be, and not change interface outside @yiguolei @yangzhg -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
