Dinesh Bhat has posted comments on this change.

Change subject: KUDU-1753 [tablet_service] continue scan request on deleted 
tablet
......................................................................


Patch Set 2:

Alexey, just thinking out loud here:  Tablet not found in the JIRA corresponds 
to tablet tombstoned state in the tablet server, which means any subsequent 
scan on that server is not expected to yield any data, right ? This fix aims to 
continue the scan until it fetches whatever is cached so far. So, I am curious 
if this guarantees to fix the underlying issue though, because the GetTabletRef 
can fail anywhere during the scan and not just under HandleContinueScanRequest, 
right ?

-- 
To view, visit http://gerrit.cloudera.org:8080/5346
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ica48c52a81862f47a9245003915d18be411bf8b1
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com>
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-HasComments: No

Reply via email to