Github user blrunner commented on a diff in the pull request:
https://github.com/apache/tajo/pull/653#discussion_r37882457
--- Diff:
tajo-catalog/tajo-catalog-common/src/main/proto/CatalogProtos.proto ---
@@ -247,13 +240,15 @@ message PartitionDescProto {
required string partitionName = 1;
repeated PartitionKeyProto partitionKeys = 2;
optional string path = 3;
- optional int32 id = 4;
+ optional int32 partition_id = 4;
--- End diff --
PartitionDescProto is used for various catalog API, for examples, output
parameter of getPartitionByPartitionName and getPartitionsByTableName and
getAllPartitions, input parameter of alterTable and addPartitions. In case of
output parameter, partition id always is not null. But in case of input
parameter, partition id will be null because CatalogClient can get partition
informs with table name and partition name.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
