Skip to content

Commit

Permalink
Merge pull request #3 from gojektech/empty_result_handling
Browse files Browse the repository at this point in the history
Empty response handling in serving api
  • Loading branch information
pradithya authored Dec 13, 2018
2 parents f15a6a7 + ee7fd87 commit 1b1bd2b
Showing 1 changed file with 0 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -77,14 +77,6 @@ public void queryFeatures(Request request, StreamObserver<Response> responseObse
Request validRequest = checkTimestampRange(request);
Response response = feast.queryFeatures(validRequest);

if (response.getEntitiesCount() < 1) {
responseObserver.onError(
new StatusRuntimeException(
Status.fromCode(NOT_FOUND)
.withDescription("No value is found for the requested feature")));
return;
}

innerSpan.log("calling onNext");
responseObserver.onNext(response);
innerSpan.log("calling onCompleted");
Expand Down

0 comments on commit 1b1bd2b

Please sign in to comment.