falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pallavi Rao" <pallavi....@inmobi.com>
Subject Re: Review Request 37441: Property for default number of results is not loaded dynamically
Date Fri, 14 Aug 2015 07:12:43 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37441/#review95397
-----------------------------------------------------------

Ship it!


Ship It!

- Pallavi Rao


On Aug. 14, 2015, 7:05 a.m., Ajay Yadava wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37441/
> -----------------------------------------------------------
> 
> (Updated Aug. 14, 2015, 7:05 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1399
>     https://issues.apache.org/jira/browse/FALCON-1399
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Currently the property webservices.default.results.per.page is in runtime.properties.
Properties in runtime.properties are supposed to be loaded dynamically, however chaning this
property doesn't reflect it as the code reads it once and uses the initial value forever.
> 
> 
> Diffs
> -----
> 
>   common/src/main/resources/runtime.properties f0253e6 
>   prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java ff5dc04 
>   prism/src/main/java/org/apache/falcon/resource/AbstractInstanceManager.java b382f95

>   prism/src/main/java/org/apache/falcon/resource/proxy/InstanceManagerProxy.java 465a636

>   prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java
b49d6f8 
>   src/conf/runtime.properties 1eb3f1c 
>   webapp/src/main/java/org/apache/falcon/resource/InstanceManager.java cbc936e 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java 9a3d6be

> 
> Diff: https://reviews.apache.org/r/37441/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ajay Yadava
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message