krishan1390 opened a new pull request, #5576: URL: https://github.com/apache/hadoop/pull/5576
### Description of PR Requirement is captured in JIRA ### Code Changes 1. Create a new command and rpc method for Admin Service (& the required protobufs) 2. Provide an argument to start RM in safe mode which will only start Admin Service 3. Create a generic KV interface & generic LevelDB interface to access the DB ### How was this patch tested? 1. Tested can't access the DB when RM is running. 2. Tested basic happy cases of get / set / del with valid & invalid arguments 3. Tested can access DB when RM can't start due to DB corruption - and can update DB in this case to fix RM state 4. Tested can't access when DB is not yet inited 5. Tested can't access if DB is being used by another RM process. Also RM can't start if RM is already started in safe mode & vice versa (this is because of level db locks) 6. Tested Conf store keys & RM DT Master keys Unit test cases aren't really applicable here as all code changes are about tying together various pieces from AdminCLI to LevelDB rather than any functional logic -- 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]
