bshashikant commented on pull request #1076: URL: https://github.com/apache/hadoop-ozone/pull/1076#issuecomment-645172805
> > Thank you for the patch @sadanand48. Any reason why we are not using the OMDBDefinition to actually create the OM DB, like in the SCM? > > @avijayanhwx I think the plan was to use DBStore interface instead of the MetadataDataStore and in order to do that DBDefinition must be defined [([HDDS-3172](https://issues.apache.org/jira/browse/HDDS-3172))] .([#700 (comment)](https://github.com/apache/hadoop-ozone/pull/700#issuecomment-615289480)). > OM currently uses the MetadataStore Interface so there is no OMDBDefinition defined. I think we probably need to follow the same model as that in SCM. Though the patch looks good, other changes related to using DB interface rather than MetadataDataStore interface, can be done in separate jira after further analysis. @avijayanhwx , what do you think? ---------------------------------------------------------------- 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]
