[GitHub] [incubator-pegasus] levy5307 commented on a change in pull request #825: Fix full scan

[GitBox <gi...@apache.org>]

levy5307 commented on a change in pull request #825:
URL: https://github.com/apache/incubator-pegasus/pull/825#discussion_r727779505



##########
File path: src/client_lib/pegasus_client_impl.cpp
##########
@@ -1179,7 +1179,7 @@ int pegasus_client_impl::get_scanner(const std::string &hash_key,
     if (c < 0 || (c == 0 && o.start_inclusive && o.stop_inclusive)) {
         v.push_back(pegasus_key_hash(start));
     }
-    scanner = new pegasus_scanner_impl(_client, std::move(v), o, start, stop, false);

Review comment:
       Could you add description to show what kind of bug did you encounter?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org