Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/9240 )

Change subject: Add back KuduColumnSchema DataTypeToString
......................................................................

Add back KuduColumnSchema DataTypeToString

KuduColumnSchema’s DataTypeToString method
was accidentally removed in KUDU-721.

This method/functionality should be added back
to prevent a compatibility break. However, it’s
implimentation has been updated to print based
on the public DataType instead of converting to the
internal type and printing that.

Change-Id: I2bf6879f5ab623366a962c6a3d51efc49978a53c
Reviewed-on: http://gerrit.cloudera.org:8080/9240
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <d...@cloudera.com>
---
M src/kudu/client/schema.cc
M src/kudu/client/schema.h
2 files changed, 33 insertions(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Dan Burkert: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/9240
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I2bf6879f5ab623366a962c6a3d51efc49978a53c
Gerrit-Change-Number: 9240
Gerrit-PatchSet: 7
Gerrit-Owner: Grant Henke <granthe...@gmail.com>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to