[GitHub] nifi-registry pull request #33: NIFIREG-47 Improvements to nifi-registry-cli...

2017-11-07 Thread bbende
Github user bbende closed the pull request at:

https://github.com/apache/nifi-registry/pull/33


---


[GitHub] nifi-registry pull request #33: NIFIREG-47 Improvements to nifi-registry-cli...

2017-11-06 Thread markap14
Github user markap14 commented on a diff in the pull request:

https://github.com/apache/nifi-registry/pull/33#discussion_r149114864
  
--- Diff: 
nifi-registry-client/src/main/java/org/apache/nifi/registry/client/impl/JerseyBucketClient.java
 ---
@@ -140,7 +150,8 @@ public Fields getFields() throws NiFiRegistryException, 
IOException {
 target = target.queryParam("sort", sortParam.toString());
 }
 
-return Arrays.asList(target.request().get(Bucket[].class));
+return getRequestBuilder(target).get(List.class);
--- End diff --

I think this should be done as above in the other getAll() call no? Getting 
a Bucket[] and then returning emptyList or Arrays.asList?


---