----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36758/#review92877 -----------------------------------------------------------
client/src/main/java/org/apache/falcon/cli/FalconCLI.java (line 249) <https://reviews.apache.org/r/36758/#comment147140> Generally, not a good practice to default an integer to null.. Will require additional null checks and may cause NPEs otherwise. src/conf/runtime.properties (line 37) <https://reviews.apache.org/r/36758/#comment147138> Why do we need 2 props here? Shouldn't just one do? If user does not supply numResults, the value it should default to. What is the purpose of introducing an upper limit that will override numResults? Is there a system limitation that requires us to specify this? If not, I would prefer to keep it simple and just have one property. - Pallavi Rao On July 24, 2015, 12:46 a.m., Ajay Yadava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36758/ > ----------------------------------------------------------- > > (Updated July 24, 2015, 12:46 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1324 > https://issues.apache.org/jira/browse/FALCON-1324 > > > Repository: falcon-git > > > Description > ------- > > Currently pagination API breaks backward compatibility as it by default > returns 10 results. For example, earlier > falcon entity -type feed -list > used to return all the feeds but now the same command will return only 10 > results. > > Proposed solution: > We should make the default number of results configurable in the properties > file and put it to a high value by default so that behavior is backwards > compatible. > > > Diffs > ----- > > client/src/main/java/org/apache/falcon/cli/FalconCLI.java e393f82 > client/src/main/java/org/apache/falcon/client/FalconClient.java 555bdb7 > common/src/main/resources/runtime.properties 3b32463 > prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java > a3801e9 > prism/src/main/java/org/apache/falcon/resource/AbstractInstanceManager.java > 310e73b > > prism/src/main/java/org/apache/falcon/resource/proxy/InstanceManagerProxy.java > ac3e5db > > prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java > eb281d3 > src/conf/runtime.properties 58dee3d > webapp/src/main/java/org/apache/falcon/resource/InstanceManager.java > 9c5538b > > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java > ed6f44e > > Diff: https://reviews.apache.org/r/36758/diff/ > > > Testing > ------- > > > Thanks, > > Ajay Yadava > >
