Miklos Gergely created HIVE-20807:
-------------------------------------

             Summary: Refactor LlapStatusServiceDriver
                 Key: HIVE-20807
                 URL: https://issues.apache.org/jira/browse/HIVE-20807
             Project: Hive
          Issue Type: Improvement
          Components: Hive
    Affects Versions: 4.0.0
            Reporter: Miklos Gergely
            Assignee: Miklos Gergely
             Fix For: 4.0.0


LlapStatusServiceDriver is the class used to determine if LLAP has started. The 
following problems should be solved by refactoring:

1. The main class is more than 800 lines long,should be cut into multiple 
smaller classes.
2. The current design makes it extremely hard to write unit tests.
3. There are some overcomplicated, over-engineered parts of the code.
4. Most of the code is under org.apache.hadoop.hive.llap.cli, but some parts 
are under org.apache.hadoop.hive.llap.cli.status. The whole program could be 
moved to the latter.
5. LlapStatusHelpers serves as a class for holding classes, which doesn't make 
much sense.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to