Attila Jeges has posted comments on this change. ( )

Change subject: IMPALA-8842 part 1: (Hive3) Use 'engine' field in HMS stat API

Patch Set 7:

(1 comment)
Commit Message:
PS7, Line 14: This change is Step 2 in a series of steps to coordinate the
            : introduction of HMS API changes to Hive3 and Impala. Step 4 is 
            : Impala related, it will be covered in IMPALA-8842 part 2.
            : The steps are as follows:
            : 1. Change in Hive3.
            : We push new APIs so Impala can use them. New APIs will simply 
call old
            : existing methods so there should not be any change of 
            : there. Since there were many incompatible changes, new APIs are 
            : method_name_V2.
            : 2. Change in Impala.
            : Push changes to use new methods *V2.
            : 3. Change in Hive3.
            : Push patch with complete functionality. *V2 methods contains the 
            : logic. The old existing methods are not used anymore by Impala at 
            : point, hence they can be removed. For every method_name_V2, I will
            : create a corresponding method method_name that calls the former 
            : 4. Change in Impala
            : Replace *V2 calls by *.
            : 5. Change in Hive3.
            : Remove *V2 methods from API
> I recorded the steps in the commit msg because I wanted to explain why we n
I also wanted to explain why there's no change in functionality yet.

To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a73f5eeac8e84d63b22aaed5dfbcd8ea39f0af4
Gerrit-Change-Number: 14043
Gerrit-PatchSet: 7
Gerrit-Owner: Attila Jeges <>
Gerrit-Reviewer: Attila Jeges <>
Gerrit-Reviewer: Csaba Ringhofer <>
Gerrit-Reviewer: Impala Public Jenkins <>
Gerrit-Reviewer: Vihang Karajgaonkar <>
Gerrit-Comment-Date: Thu, 15 Aug 2019 18:01:04 +0000
Gerrit-HasComments: Yes

Reply via email to