hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Enis Soztutar (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-18507) [C++] Support for MultiPuts in AsyncBatchRpcRetryingCaller class
Date Sat, 12 Aug 2017 00:24:00 GMT

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

Enis Soztutar commented on HBASE-18507:

Thanks for the updating the work with templates. 

- We have updated the base folly version, these will not compile anymore: 
-#include <folly/Try.h>
 #include <folly/futures/Future.h>
 #include <folly/futures/Promise.h>
+#include <folly/futures/Try.h>
-#include <folly/Unit.h>
-#include <folly/futures/Future.h>
+#include <folly/futures/Unit.h>
(you can update the folly version using the same exact version, and commands from {{docker-files/Dockerfile}},
see section for folly and wangle. 
- The patch undo's the changes already committed from HBASE-18564 (especially client-test.cc).
Can you please only do changes related to the new patch. 
- remove: 
+  LOG(INFO) << "Does this work";
+  for (const auto& row : rows) {
+    VLOG(4) << "row is " << row->row();
+  }
- Nice tests, we can add more like the ones in multi-retry-test. 
- Are you gonna address this? 
+  // TODO pass the below as parameter
+  // connection_conf_->read_rpc_timeout();
+  // connection_conf_->write_rpc_timeout();
- Let's throw an exception here: 
+          //TODO runtime_error reqd ?
also this should not be caught, or should be rethrown: 
+      } catch (const std::bad_typeid &e) {
+        std::cout << " caught " << e.what() << '\n';
+      }
- This does not look right:
if (strstr(typeid(*pget).name(), "Get") != nullptr) 
why are you doing string comparison. It is very inefficient to do this. We should just compare
the typeids, or templatize this code path as well.
- Remove if not needed: 
+// template class RequestConverter<hbase::Row>;
+// template class AsyncBatchRpcRetryingCaller<folly::Unit>;
- We have talked (offline) about Puts returning {{folly::Unit}} instead of empty {{Result}}
objects. Are you gonna address that?
+folly::Future<std::vector<folly::Try<std::shared_ptr<Result>>>> RawAsyncTable::Put(
- Remaining looks good. 

> [C++] Support for MultiPuts in AsyncBatchRpcRetryingCaller class
> ----------------------------------------------------------------
>                 Key: HBASE-18507
>                 URL: https://issues.apache.org/jira/browse/HBASE-18507
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Sudeep Sunthankar
>            Assignee: Sudeep Sunthankar
>         Attachments: HBASE-18507.HBASE-14850.v1.patch, HBASE-18507.HBASE-14850.v2.patch
> We will be addressing Multi Puts changes to AsyncBatchRpcRetryingCaller class here

This message was sent by Atlassian JIRA

View raw message