You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Yingchun Lai (Code Review)" <ge...@cloudera.org> on 2021/10/18 05:49:07 UTC

[kudu-CR] [master] Validate extra config properties for a table

Hello Alexey Serbin, Kudu Jenkins, Bankim Bhavsar, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/17914

to look at the new patch set (#3).

Change subject: [master] Validate extra config properties for a table
......................................................................

[master] Validate extra config properties for a table

Prior to this patch, no validation was performed on the strings
representing the keys of extra configuration properties for a
table.
This patch adds the missing validation logic, so now an error
is returned on attempt to set unsupported extra configuration
properties for a table.

Change-Id: Iff5ccc553fdbe164cf342df6882ea0b78a9e10f9
---
M src/kudu/client/client-test.cc
M src/kudu/common/wire_protocol.cc
2 files changed, 34 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/14/17914/3
-- 
To view, visit http://gerrit.cloudera.org:8080/17914
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iff5ccc553fdbe164cf342df6882ea0b78a9e10f9
Gerrit-Change-Number: 17914
Gerrit-PatchSet: 3
Gerrit-Owner: Yingchun Lai <ac...@gmail.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Bankim Bhavsar <ba...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <ac...@gmail.com>