[GitHub] [hive] hsnusonic commented on a change in pull request #2073: HIVE-24828: Provide new HMS API to return latest committed compaction…

[GitBox <gi...@apache.org>]

hsnusonic commented on a change in pull request #2073:
URL: https://github.com/apache/hive/pull/2073#discussion_r595492829



##########
File path: standalone-metastore/metastore-common/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/AddDynamicPartitions.java
##########
@@ -787,13 +787,13 @@ public void read(org.apache.thrift.protocol.TProtocol iprot, AddDynamicPartition
           case 5: // PARTITIONNAMES
             if (schemeField.type == org.apache.thrift.protocol.TType.LIST) {
               {
-                org.apache.thrift.protocol.TList _list828 = iprot.readListBegin();
-                struct.partitionnames = new java.util.ArrayList<java.lang.String>(_list828.size);
-                @org.apache.thrift.annotation.Nullable java.lang.String _elem829;
-                for (int _i830 = 0; _i830 < _list828.size; ++_i830)

Review comment:
       I've verified the thrift version is correct and I can compile it correctly on the current master branch. These changes are generated after I introduced three new structs.




----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org