[Geoserver-devel] Reverting a change to fix the build

Hi,
a quick note to report I rolled back the commit at revision 9856
by David, reading:

Log Message
Use resourcepool for FeatureTypeInfoImpl's AttributeInfo list.
Modified Paths
trunk/geoserver/main/src/main/java/org/geoserver/catalog/ResourcePool.java
trunk/geoserver/main/src/main/java/org/geoserver/catalog/impl/FeatureTypeInfoImpl.java

This change broke the build, investigation showed the attributes
for various types were not retained anymore, leading to failures
in srs handling (it checks there is any attribute before trying
to perform any srs forcing/reprojection):
http://gridlock.openplans.org:8080/hudson/job/geoserver-trunk/430/console

I'm not sure what the meaning of the commit is either,
FeatureTypeInfo is held by the resource pool, and it
holds the AttributeTypeInfo, so the resource pool is
already keeping a tab onto the AttributeTypeInfo indirectly,
I don't get the reason why we need a direct holding.

To recap, sorry for reverting your work, I'm curious
to learn why the change was needed, ping me when you
need to fix this back, I'll try helping you to bring
back the change.

Cheers
Andrea