Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-17 Thread Gabriel Roldan
fair enough, good for me then. Gabriel Andrea Aime wrote: > Justin Deoliveira ha scritto: >> Thanks Gabriel. I replied to the feedback. >> >> I don't have too strong a preference here about making the MetadataMap >> a map or not. I would lean toward keeping it a map, but i could be >> persuaded

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-17 Thread Andrea Aime
Justin Deoliveira ha scritto: > Thanks Gabriel. I replied to the feedback. > > I don't have too strong a preference here about making the MetadataMap a > map or not. I would lean toward keeping it a map, but i could be > persuaded the other way if your preference is a strong one? > > Anyways, m

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-17 Thread Justin Deoliveira
Thanks Gabriel. I replied to the feedback. I don't have too strong a preference here about making the MetadataMap a map or not. I would lean toward keeping it a map, but i could be persuaded the other way if your preference is a strong one? Anyways, maybe Andrea can be a tie breaker. Gabriel R

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-16 Thread Gabriel Roldan
Justin Deoliveira wrote: > I have gone and implemented the map as based on this thread. I would > love a review: I've posted my feedback on the issue. Basically it looks good, I just keep on thinking there's no need for MetadataMap to implementa java.util.Map. Proposed change being it either ju

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-16 Thread Justin Deoliveira
I have gone and implemented the map as based on this thread. I would love a review: http://jira.codehaus.org/browse/GEOS-3164 -Justin Gabriel Roldan wrote: > Andrea Aime wrote: >> Gabriel Roldan ha scritto: >>> Andrea Aime wrote: Justin Deoliveira ha scritto: > Hi all, > > A fe

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-12 Thread Justin Deoliveira
Thanks for your input guys, I think this is a really good idea. I will come up with a patch soon for review. -Justin Gabriel Roldan wrote: > Andrea Aime wrote: >> Gabriel Roldan ha scritto: >>> Andrea Aime wrote: Justin Deoliveira ha scritto: > Hi all, > > A few issues have popp

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-12 Thread Andrea Aime
Gabriel Roldan ha scritto: > Andrea Aime wrote: >> Justin Deoliveira ha scritto: >>> Hi all, >>> >>> A few issues have popped up recently related to how maps (in >>> particular the metadata map for catalog objects) is >>> persisted/depersisted. The latest issue revolves around someone >>> trying

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-12 Thread Gabriel Roldan
Andrea Aime wrote: > Justin Deoliveira ha scritto: >> Hi all, >> >> A few issues have popped up recently related to how maps (in particular >> the metadata map for catalog objects) is persisted/depersisted. The >> latest issue revolves around someone trying ot configure a feature type >> via res

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-12 Thread Gabriel Roldan
Andrea Aime wrote: > Gabriel Roldan ha scritto: >> Andrea Aime wrote: >>> Justin Deoliveira ha scritto: Hi all, A few issues have popped up recently related to how maps (in particular the metadata map for catalog objects) is persisted/depersisted. The latest issue revolve

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-12 Thread Andrea Aime
Justin Deoliveira ha scritto: > Hi all, > > A few issues have popped up recently related to how maps (in particular > the metadata map for catalog objects) is persisted/depersisted. The > latest issue revolves around someone trying ot configure a feature type > via rest. > > The issue is that

Re: [Geoserver-devel] xstream persistence and metadata map

2009-06-11 Thread Ben Caradoc-Davies
Justin Deoliveira wrote: > So I am wondering. What do people think about making it practice that > whenever accessing data from a metadata map, that a converter be used. > The downside of course is that we have to update a bunch of code, and > its one more thing to remember. The upside is that t

[Geoserver-devel] xstream persistence and metadata map

2009-06-11 Thread Justin Deoliveira
Hi all, A few issues have popped up recently related to how maps (in particular the metadata map for catalog objects) is persisted/depersisted. The latest issue revolves around someone trying ot configure a feature type via rest. The issue is that in an attempt to make xstream less verbose, i