+1 on Query.getAttributes():List<PropertyName> Sorry I wasn't clear enough. On Fri, 2010-12-24 at 16:32 +1100, Jody Garnett wrote: > To which idea is this +1 directed :-) > > I am getting a bit nervous watching PropertyName being used as a placeholder > for a generic "qualified" XPath. Still I guess I understand why it is taking > that shape. > > So is your +1 for the idea of: > > Query.getAttributes(): List<PropertyName> // property name may refer to a > complete xpath > > With this in mind getPropertyNames and setPropertyNames would then become a > shortcut methods for the above List. > > Jody > > On 23/12/2010, at 11:51 PM, Gabriel Roldán wrote: > > > +1 > > > > On Mon, 2010-12-20 at 17:23 +1100, Jody Garnett wrote: > >> I though about this; like the idea of using a List<Name> from an ease > >> of use / consistency point of view. However I think I may of slightly > >> missed the point ... > >> > >> > >> In the code example I tried to show the use of an xpath for > >> propertyNames so you could indicate how much of contained data > >> structures you wanted filled out. > >> > >> > >> query.setPropertyNames( new > >> String[]{"foo:x","foo:y","foo:description/foo:label"}); > >> > >> > >> The idea was that a "description" object would be created; but only > >> the "label" field filled in with a non null result. > >> This kind of thing we cannot obtain using a List<Name>; but would be > >> possible with a List<PropertyName> > >> > >> > >> Jody > >> > >> > >> On 20/12/2010, at 4:10 PM, Niels wrote: > >> > >>> On 18/12/10 17:12, Andrea Aime wrote: > >>>> On Fri, Dec 17, 2010 at 9:03 AM, Niels<niels.charl...@csiro.au> > >>>> wrote: > >>>>> I think you do need a NameSpace context in your query, because > >>>>> you cannot > >>>>> assume that the namespace prefixes in the property names of the > >>>>> query are > >>>>> the same as the ones in the mapping or in the datastore or > >>>>> anywhere else. > >>>>> The thing about namespace prefixes is that you can never make > >>>>> those > >>>>> assumptions. > >>>>> Another solution for Query would be to provide a list of > >>>>> PropertyNames > >>>>> rather than strings (which is what I suggested earlier I think?) > >>>>> but I guess > >>>>> that would break too much existing code ? > >>>> Lucklily Query is not an interface anymore so it _would_ be > >>>> possible to > >>>> just add PropertyName (or Name) support and have it fall back on > >>>> plain > >>>> strings when the plain string methods are called. The new methods > >>>> would be used only by code that know about complex features. > >>> > >>> I think that is an excellent idea actually... Have two getters and > >>> settesr for PropertyNames property in Query: one with string[] and > >>> one > >>> with PropertyName[]. The inner representation of the property names > >>> in > >>> the class could be PropertyName[], but when the string[] getter and > >>> setter are used, conversion is done automatically. > >>> > >>> I think that would be even better than having a > >>> setNamespaceContext() in > >>> Query! > >>> > >>> Niels > >>> > >>> ------------------------------------------------------------------------------ > >>> Lotusphere 2011 > >>> Register now for Lotusphere 2011 and learn how > >>> to connect the dots, take your collaborative environment > >>> to the next level, and enter the era of Social Business. > >>> http://p.sf.net/sfu/lotusphere-d2d > >>> _______________________________________________ > >>> Geotools-devel mailing list > >>> Geotools-devel@lists.sourceforge.net > >>> https://lists.sourceforge.net/lists/listinfo/geotools-devel > >>> > >> > >> > >> > >> > >> On 20/12/2010, at 4:10 PM, Niels wrote: > >> > >>> On 18/12/10 17:12, Andrea Aime wrote: > >>>> On Fri, Dec 17, 2010 at 9:03 AM, Niels<niels.charl...@csiro.au> > >>>> wrote: > >>>>> I think you do need a NameSpace context in your query, because > >>>>> you cannot > >>>>> assume that the namespace prefixes in the property names of the > >>>>> query are > >>>>> the same as the ones in the mapping or in the datastore or > >>>>> anywhere else. > >>>>> The thing about namespace prefixes is that you can never make > >>>>> those > >>>>> assumptions. > >>>>> Another solution for Query would be to provide a list of > >>>>> PropertyNames > >>>>> rather than strings (which is what I suggested earlier I think?) > >>>>> but I guess > >>>>> that would break too much existing code ? > >>>> Lucklily Query is not an interface anymore so it _would_ be > >>>> possible to > >>>> just add PropertyName (or Name) support and have it fall back on > >>>> plain > >>>> strings when the plain string methods are called. The new methods > >>>> would be used only by code that know about complex features. > >>> > >>> I think that is an excellent idea actually... Have two getters and > >>> settesr for PropertyNames property in Query: one with string[] and > >>> one > >>> with PropertyName[]. The inner representation of the property names > >>> in > >>> the class could be PropertyName[], but when the string[] getter and > >>> setter are used, conversion is done automatically. > >>> > >>> I think that would be even better than having a > >>> setNamespaceContext() in > >>> Query! > >>> > >>> Niels > >>> > >>> ------------------------------------------------------------------------------ > >>> Lotusphere 2011 > >>> Register now for Lotusphere 2011 and learn how > >>> to connect the dots, take your collaborative environment > >>> to the next level, and enter the era of Social Business. > >>> http://p.sf.net/sfu/lotusphere-d2d > >>> _______________________________________________ > >>> Geotools-devel mailing list > >>> Geotools-devel@lists.sourceforge.net > >>> https://lists.sourceforge.net/lists/listinfo/geotools-devel > >>> > >> > >> ------------------------------------------------------------------------------ > >> Lotusphere 2011 > >> Register now for Lotusphere 2011 and learn how > >> to connect the dots, take your collaborative environment > >> to the next level, and enter the era of Social Business. > >> http://p.sf.net/sfu/lotusphere-d2d > >> _______________________________________________ Geotools-devel mailing > >> list Geotools-devel@lists.sourceforge.net > >> https://lists.sourceforge.net/lists/listinfo/geotools-devel > > > > -- > > Gabriel Roldan > > grol...@opengeo.org > > Expert service straight from the developers > > >
-- Gabriel Roldan grol...@opengeo.org Expert service straight from the developers ------------------------------------------------------------------------------ Learn how Oracle Real Application Clusters (RAC) One Node allows customers to consolidate database storage, standardize their database environment, and, should the need arise, upgrade to a full multi-node Oracle RAC database without downtime or disruption http://p.sf.net/sfu/oracle-sfdevnl _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel