I had a look, but have a hesitation accepting a pull request when I am not in
position to test (against oracle for SYNONYM).
I have just finished my pre-flight checks for GeoTools, but would like to see
if we can get this in since you have asked so nicely. While we wait to see if
an active JDBC developer is in position to review I will start a build locally
to confirm your change "works for me".
--
Jody Garnett
On Wednesday, 17 July 2013 at 5:23 PM, Robin Pradel wrote:
> Hi developers with permission to accept/apply pull requests,
>
> Can somebody have a look at the pull requests mentioned below? These ones
> solve the issue about synonyms in databases
> http://jira.codehaus.org/browse/GEOT-4486
>
> Andrea had a look at the code in a previous pull request which was ok with a
> small test case fix (but on closed branch).
>
> Yours sincerely.
>
> Robin Pradel
> System Ingenieur
> IVU Traffic Technologies AG
> Bundesallee 88, D-12161 Berlin
> http://www.ivu.de (http://www.ivu.de/)
>
>
>
> Von: andrea.a...@gmail.com [mailto:andrea.a...@gmail.com] Im Auftrag von
> Andrea Aime
> Gesendet: Montag, 15. Juli 2013 14:50
> An: Robin Pradel
> Cc: geotools-devel@lists.sourceforge.net
> (mailto:geotools-devel@lists.sourceforge.net)
> Betreff: Re: [Geotools-devel] Preparing for the release
>
> On Mon, Jul 15, 2013 at 2:44 PM, Robin Pradel <r...@ivu.de
> (mailto:r...@ivu.de)> wrote:
> > Hi Andrea,
> >
> > I fixed the pull requests for synonyms in the JDBCDataStore:
> > Master: https://github.com/geotools/geotools/pull/227
> > 9.x: https://github.com/geotools/geotools/pull/226
> >
> > Can you apply these ones for the coming release? That would be very nice.
> >
> >
> >
> >
> >
> >
>
>
>
> Hi Robin,
>
> unfortunately I won't have time to look again at them until next weekend, the
> working
>
> week is full beyond salvation (just got back from vacations, lots of stuff
> piled up).
>
>
>
> However, there are other developers on this mailing list, maybe someone else
>
> wants to have a look?
>
>
>
> Cheers
>
> Andrea
>
>
>
>
> --
> ==
>
> Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
> information.
>
> ==
>
>
>
>
> Ing. Andrea Aime
>
> @geowolf
>
> Technical Lead
>
>
>
> GeoSolutions S.A.S.
>
> Via Poggio alle Viti 1187
>
> 55054 Massarosa (LU)
>
> Italy
>
> phone: +39 0584 962313
>
> fax: +39 0584 1660272
>
> mob: +39 339 8844549
>
>
>
> http://www.geo-solutions.it
>
> http://twitter.com/geosolutions_it
>
>
>
> -------------------------------------------------------
>
>
>
>
>
>
> ------------------------------------------------------------------------------
> See everything from the browser to the database with AppDynamics
> Get end-to-end visibility with application monitoring from AppDynamics
> Isolate bottlenecks and diagnose root cause in seconds.
> Start your free trial of AppDynamics Pro today!
> http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
>
> _______________________________________________
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> (mailto:GeoTools-Devel@lists.sourceforge.net)
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
