falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ajay Yadava" <ajayn...@gmail.com>
Subject Re: Review Request 37441: Property for default number of results is not loaded dynamically
Date Fri, 14 Aug 2015 07:05:55 GMT

-----------------------------------------------------------
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.


Changes
-------

Rebased the patch.


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 (updated)
-----

  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