[ 
https://issues.apache.org/jira/browse/HIVE-26692?focusedWorklogId=830202&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-830202
 ]

ASF GitHub Bot logged work on HIVE-26692:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 30/Nov/22 19:49
            Start Date: 30/Nov/22 19:49
    Worklog Time Spent: 10m 
      Work Description: asolimando commented on code in PR #3820:
URL: https://github.com/apache/hive/pull/3820#discussion_r1036382446


##########
pom.xml:
##########
@@ -1773,6 +1773,35 @@
       <id>thriftif</id>
       <build>
         <plugins>
+          <plugin>
+            <groupId>org.codehaus.mojo</groupId>
+            <artifactId>exec-maven-plugin</artifactId>
+            <version>1.2.1</version>

Review Comment:
   Thanks Ayush for noticing this, unfortunately the `standalone-metastore` pom 
does not inherit from the topmost one, so we can't share it, but I will 
introduce a variable with the same name and use the same version.
   
   I have tried locally, everything seems good (normal build and thrift 
generation, OK and KO paths).





Issue Time Tracking
-------------------

    Worklog Id:     (was: 830202)
    Time Spent: 50m  (was: 40m)

> Check for the expected thrift version before compiling
> ------------------------------------------------------
>
>                 Key: HIVE-26692
>                 URL: https://issues.apache.org/jira/browse/HIVE-26692
>             Project: Hive
>          Issue Type: Task
>          Components: Thrift API
>    Affects Versions: 4.0.0-alpha-2
>            Reporter: Alessandro Solimando
>            Assignee: Alessandro Solimando
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> At the moment we don't check for the thrift version before launching thrift, 
> the error messages are often cryptic upon mismatches.
> An explicit check with a clear error message would be nice, like what parquet 
> does: 
> [https://github.com/apache/parquet-mr/blob/master/parquet-thrift/pom.xml#L247-L268]
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to