[ 
https://issues.apache.org/jira/browse/HBASE-14006?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15862769#comment-15862769
 ] 

Duo Zhang commented on HBASE-14006:
-----------------------------------

After reading the doc of thrift, I think it is allowed to add optional args to 
a function call as the parameters of a function call will be implemented as a 
struct.

But you should not change the old field identifier, i.e., please keep the old 
field identifier '4' for the 'attributes' parameter. Your new 'numVersions' 
should use '5' as its field identifier.

Thanks.

> Hbase Thrift - add num versions to rowWithColumns
> -------------------------------------------------
>
>                 Key: HBASE-14006
>                 URL: https://issues.apache.org/jira/browse/HBASE-14006
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: navdeep agrawal
>            Assignee: Zhang Bo
>            Priority: Critical
>         Attachments: HBASE-14006.patch
>
>
> in thrift if we want column map and also want to pass num versions ,we dont 
> have any function for that . ie no numversion parameter in rowWithColumn and 
> rowsWithColumn methods .
> we can have a function which have columns map and numbversions parameter



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to