Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17704
Change subject: [common] 'client_schema' isn't needed in PartitionSchema::FromPB() ...................................................................... [common] 'client_schema' isn't needed in PartitionSchema::FromPB() This patch removes unnecessary 'client_schema' parameter introduced for PartitionSchema::FromPB() recently, also removing the overloaded variant of the PartitionSchema::FromPB() method as well. As far as I can see, neither ApplyAlterPartitioningSteps() nor CreateTable() methods of the CatalogManager class needs to pass client schema to PartitionSchema::FromPB(), and the tablet schema is a must in both places anyway because column identifiers are required there. Moreover, passing the client schema would only make the decoding slower because of the logic to loop over columns while building projection mapping in RowOperationsPBDecoder::DecodeOperations(). This is a follow-up to 2dda869456659a36247eb89f5b9e5e3837e5f8a3. Change-Id: Iff5697bc51046052da13c758d79d8c659bc90711 --- M src/kudu/common/partition.cc M src/kudu/common/partition.h M src/kudu/common/schema.cc M src/kudu/master/catalog_manager.cc 4 files changed, 8 insertions(+), 19 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/17704/1 -- To view, visit http://gerrit.cloudera.org:8080/17704 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Iff5697bc51046052da13c758d79d8c659bc90711 Gerrit-Change-Number: 17704 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
