Dimitris Tsirogiannis has posted comments on this change. ( http://gerrit.cloudera.org:8080/8851 )
Change subject: IMPALA-3193: Show table's comment on show tables ...................................................................... Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/8851/9/be/src/catalog/catalog-server.cc File be/src/catalog/catalog-server.cc: http://gerrit.cloudera.org:8080/#/c/8851/9/be/src/catalog/catalog-server.cc@368 PS9, Line 368: if (table_info.__isset.comment) { : Value table_comment(table_info.comment.c_str(), document->GetAllocator()); : table_obj.AddMember("comment", table_comment, document->GetAllocator()); : } Did you verify that this shows up in the web UI of impalad and catalogd? I don't remember seeing any changes in www/catalog.tmpl. -- To view, visit http://gerrit.cloudera.org:8080/8851 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I44f814af05db6f3c027718ade9f474f8b8153bcd Gerrit-Change-Number: 8851 Gerrit-PatchSet: 9 Gerrit-Owner: Kim Jin Chul <jinc...@gmail.com> Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogian...@cloudera.com> Gerrit-Reviewer: Kim Jin Chul <jinc...@gmail.com> Gerrit-Comment-Date: Tue, 30 Jan 2018 01:57:19 +0000 Gerrit-HasComments: Yes