[ https://issues.apache.org/jira/browse/HBASE-18402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16093287#comment-16093287 ]
Ted Yu edited comment on HBASE-18402 at 7/19/17 3:45 PM: --------------------------------------------------------- lgtm was (Author: yuzhih...@gmail.com): For ThriftUtilities, this line (277) is not governed by in.isSetDeleteType() : {code} out.addColumn(column.getFamily(), column.getQualifier(), column.getTimestamp()); {code} Looks like the above is removed. Please keep non-delete logic. > Thrift2 should support DeleteFamilyVersion type > ------------------------------------------------ > > Key: HBASE-18402 > URL: https://issues.apache.org/jira/browse/HBASE-18402 > Project: HBase > Issue Type: Bug > Components: Thrift > Affects Versions: 2.0.0-alpha-1 > Reporter: Zheng Hu > Assignee: Zheng Hu > Attachments: HBASE-18402.v1.patch > > > Currently, our thrift2 only support two delete types, Actually, there are > four delete types.and we should support the other delete type: DeleteFamily > and DeleteFamilyVersion. > {code} > /** > * Specify type of delete: > * - DELETE_COLUMN means exactly one version will be removed, > * - DELETE_COLUMNS means previous versions will also be removed. > */ > enum TDeleteType { > DELETE_COLUMN = 0, > DELETE_COLUMNS = 1 > } > {code} -- This message was sent by Atlassian JIRA (v6.4.14#64029)