NihalJain commented on PR #5323: URL: https://github.com/apache/hbase/pull/5323#issuecomment-1680428986
@Apache9 Thanks for the quick response. > So if you have already tested the patch on your deployment and it does work, I'm OK with merging the current PR first. Yes this has been tested E2E in our deployment along with its dependency: ambari. > But let's open an issue for approach 1 first so we will not forget to do it. Sure let me create a JIRA for the same for follow up work. Also will update this PR with the review comments' fix as another commit here. -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org