Hello 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 (#2).
Change subject: [master] Expose error when set an unsupported table extra config
......................................................................
[master] Expose error when set an unsupported table extra config
Now table extra configs are set by strings, there is not check on
both client and server sides, a typo is easy happened without any
warnings.
This patch adds a check, an error be returned if setting an
unsupported extra config to 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/2
--
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: 2
Gerrit-Owner: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
