Re: [Geotools-devel] SLDParser and internal usage of FilterFactory(2) & ExpressionDOMParser

2021-05-25 Thread Jody Garnett
Frank here is a propsal https://github.com/geotools/geotools/wiki/Simplify-FilterFactory if you think this is something we can work on. While writing this up I found the original proposal to clean up the difference between the two

Re: [Geotools-devel] SLDParser and internal usage of FilterFactory(2) & ExpressionDOMParser

2021-05-25 Thread Jody Garnett
Frank I expect this is a case of not being aggressive enough with updating interfaces in GeoTools. When FilterFactory2 was added we were very cautious about updating interfaces because of GeoAPI approval/review/feedback. Since that time we have added additional methods (such as the temporal

[Geotools-devel] GeoTools / GeoServer PMC meeting 2021-05-25

2021-05-25 Thread Jody Garnett
GeoTools / GeoServer PMC meeting - 2021-05-25Attending Andrea Aime Jukka Rahkonnen Kevin Smith Torben Barsballe Jody Garnett Actions from last meeting: - TODO: Kevin is going to fix gwc release.rb using xpath or something - Kevin will look at GWC website upload - Jody:

Re: [Geotools-devel] SLDParser and internal usage of FilterFactory(2) & ExpressionDOMParser

2021-05-25 Thread Frank Gasdorf via GeoTools-Devel
> The class has a setter, which would have to be removed (instead of just being deprecated). > That's not our normal API removal cycle (first deprecate, then remove), but it is allowed in exceptional cases. > In this case I don't see a valid enough reason to break downstream apps that might be