search

nassim-git / project-voldemort

Automatically exported from code.google.com/p/project-voldemort
Apache License 2.0
0 stars 0 forks source link

AdminClient.getAsyncRequestList bug #348

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
In master branch, src/java/voldemort/client/protocol/admin/AdminClient.java, 
line 860

    public List<Integer> getAsyncRequestList(int nodeId, boolean showComplete) {
        VAdminProto.AsyncOperationListRequest asyncOperationListRequest = VAdminProto.AsyncOperationListRequest.newBuilder()
                                                                                                               .setShowComplete(showComplete)
                                                                                                               .build();

but in src/proto/voldemort-admin.proto line 115
message AsyncOperationListRequest {
  required int32 request_id = 1;
  required bool show_complete = 2 [default = false];
}

The request_id is required for AsyncOperationListRequest, but not set in the 
message composition.

Original issue reported on code.google.com by panxiaoz...@gmail.com on 12 May 2011 at 1:46

GoogleCodeExporter commented 9 years ago 
Fixed in my rebalance branch which will be merged to master soon...

Original comment by rsumb...@gmail.com on 20 May 2011 at 2:22