Noremac201 opened a new pull request #1747:
URL: https://github.com/apache/hive/pull/1747


   ### What changes were proposed in this pull request?
   This change is built on top of #1741 #1740.
   
   This extracts out the logic of starting a thrift hive metastore server into 
a separate class, which will allow for more modular support of additional 
listeners in the future, while also cleaning up the current hivemetastore.java 
file.
   
   ### Why are the changes needed?
   This will allow for more modular support of additional listeners in the 
future (such as grpc) since now there is a common interface to implement that 
isn't based around thrift solely.
   
   ### Does this PR introduce _any_ user-facing change?
   no
   
   ### How was this patch tested?
   Existing unit tests should cover the changes made here, since this is a 
refactoring.
   


----------------------------------------------------------------
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]

Reply via email to