1. In makePropertyResourceBundle(...) method reader is created twice: once by calling toReader(...) method, the second time in inline code
2. In the same method as 1 - shouldn't writer be closed in final block?
3. openConnectionForUrl(...) uses URLConnection#setDefaultUseCaches(...) affecting all URLConnection being created. There's setUseCaches(....) specific for this connection object