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

Enis Soztutar commented on HBASE-18174:
---------------------------------------

- Can we do the version of checkAndPut that takes the compareOp? This version 
just delegates to the one that uses equals. We can keep both overloaded 
versions. 
{code}
+  folly::Future<folly::Unit> CheckAndPut(const std::string &row, const 
std::string &family,
+          const std::string &qualifier, const std::string &value, const 
hbase::Put &put);
{code}
- This should be named {{CheckAndPutToMutateRequest}}, because we will also do 
CheckAndMutate, CheckAndDelete, etc. 
{code}
RequestConverter::CheckToMutateRequest
{code}
- Maybe name this {{ProcessedFromMutateResponse}}. 
{code}
BoolFromMutateResponse
{code}
- These lines:
{code}
auto pb_msg = 
std::static_pointer_cast<hbase::pb::MutateRequest>(checkReq->req_msg());
+                  pb::RegionSpecifier *region_specifier = 
pb_msg->mutable_region();
+                  region_specifier->set_type(
+                          
pb::RegionSpecifier_RegionSpecifierType::RegionSpecifier_RegionSpecifierType_REGION_NAME);
+                  region_specifier->set_value(region_name);
{code}
should go inside the CheckToMutateRequest, no? 


> Implement Table#checkAndPut()
> -----------------------------
>
>                 Key: HBASE-18174
>                 URL: https://issues.apache.org/jira/browse/HBASE-18174
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Ted Yu
>         Attachments: 18174.v1.txt, 18174.v7.lambda.txt, 18174.v9.txt
>
>
> This task is to implement Table#checkAndPut() method



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

Reply via email to