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

Reply via email to