Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18903 )
Change subject: KUDU-3379 Add column comments to table describe ...................................................................... Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/18903/2/src/kudu/common/schema.h File src/kudu/common/schema.h: http://gerrit.cloudera.org:8080/#/c/18903/2/src/kudu/common/schema.h@278 PS2, Line 278: const I'm curious why const is needed here? http://gerrit.cloudera.org:8080/#/c/18903/2/src/kudu/common/schema.cc File src/kudu/common/schema.cc: http://gerrit.cloudera.org:8080/#/c/18903/2/src/kudu/common/schema.cc@473 PS2, Line 473: = col_mode | nit: would '=|' operator work here automatically or it's necessary to define it? http://gerrit.cloudera.org:8080/#/c/18903/2/src/kudu/common/schema.cc@479 PS2, Line 479: ( nit: drop the extra parentheses http://gerrit.cloudera.org:8080/#/c/18903/2/src/kudu/tools/kudu-admin-test.cc File src/kudu/tools/kudu-admin-test.cc: http://gerrit.cloudera.org:8080/#/c/18903/2/src/kudu/tools/kudu-admin-test.cc@2106 PS2, Line 2106: KuduSchema schema; : : // Build the schema : { : KuduSchemaBuilder builder; : builder.AddColumn("foo")->Type(KuduColumnSchema::INT32)->NotNull(); : builder.AddColumn("bar")->Type(KuduColumnSchema::INT32)->NotNull() : ->Comment("comment for bar"); : builder.SetPrimaryKey({"foo"}); : ASSERT_OK(builder.Build(&schema)); : } : : // Create the table : { : unique_ptr<KuduTableCreator> table_creator(client_->NewTableCreator()); : ASSERT_OK(table_creator->table_name(kTableName) : .schema(&schema) : .set_range_partition_columns({"foo"}) : .num_replicas(1) : .set_owner("alice") : .Create()); : } nit: move all this under the same sub-scope? -- To view, visit http://gerrit.cloudera.org:8080/18903 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id8b751bd821a5f50490caa6f1aaf1fc767de0222 Gerrit-Change-Number: 18903 Gerrit-PatchSet: 2 Gerrit-Owner: Marton Greber <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Thu, 25 Aug 2022 03:46:16 +0000 Gerrit-HasComments: Yes
