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

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

                Author: ASF GitHub Bot
            Created on: 05/Aug/22 12:07
            Start Date: 05/Aug/22 12:07
    Worklog Time Spent: 10m 
      Work Description: ayushtkn commented on code in PR #3498:
URL: https://github.com/apache/hive/pull/3498#discussion_r938747643


##########
data/files/datasets/src_hbase/load.hive.sql:
##########
@@ -1,3 +1,5 @@
+DROP table if exists src_hbase_tmp;

Review Comment:
   Yeps, I am not sure, but my test failed once and in the next iteration when 
I fixed it, It started giving me AlreadyExist exception for the table.
   So, I added this. It doesn't occur on successive run when the test passes. I 
thought better to have this. I can try removing it, If you feel so



##########
hbase-handler/src/java/org/apache/hadoop/hive/hbase/ResultWritable.java:
##########
@@ -26,8 +26,8 @@
 import org.apache.hadoop.hbase.KeyValue;
 import org.apache.hadoop.hbase.KeyValueUtil;
 import org.apache.hadoop.hbase.client.Result;
-import org.apache.hadoop.hbase.protobuf.ProtobufUtil;
-import org.apache.hadoop.hbase.protobuf.generated.ClientProtos;
+import org.apache.hadoop.hbase.shaded.protobuf.ProtobufUtil;
+import org.apache.hadoop.hbase.shaded.protobuf.generated.ClientProtos;

Review Comment:
   The Hbase tests were failing because they had one protobuf 2.5.0 reference 
for some use case. So bugged the HBase folks and got that as a solution.
   
https://issues.apache.org/jira/browse/HBASE-21587?focusedCommentId=17575116&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17575116





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

    Worklog Id:     (was: 798408)
    Time Spent: 3h 40m  (was: 3.5h)

> Bump protobuf version to 3.21.1 (aarch64 support)
> -------------------------------------------------
>
>                 Key: HIVE-24483
>                 URL: https://issues.apache.org/jira/browse/HIVE-24483
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Cameron Moberg
>            Assignee: Ayush Saxena
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 3h 40m
>  Remaining Estimate: 0h
>
> The following protoc version's used in hive is very old i.e. 2.5.0 
> [https://repo.maven.apache.org/maven2/com/google/protobuf/protoc/] .  The 
> v2.5.0 does not have aarch64 support. But the AArch64 support started from 
> v3.5.0 on-words in google's protobuf project release. 



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

Reply via email to