Hi Jody

I guess a shove was needed to figure it out.  The new (sorry) PR is:
https://github.com/geotools/geotools/pull/2824

Thanks

Peter

On Mon, 2 Mar 2020 at 20:09, Jody Garnett <[email protected]> wrote:

> What assistance is needed here?
>
> I expect adding a table during setup, dropping it during setup (incase of
> failed run) and shutdown, and then making some methods to ensure  resulting
> feature source functions?
>
> The pull request pointed to this [class](
> https://github.com/geotools/geotools/blob/master/modules/plugin/jdbc/jdbc-postgis/src/test/java/org/geotools/data/postgis/PostGISUuidTestSetup.java),
> not sure if you want to copy that or add to it?
>
>
> --
> Jody Garnett
>
>
> On Mon, 2 Mar 2020 at 07:22, Peter Smythe <[email protected]> wrote:
>
>> Hi Jody
>>
>> I still need to write a unit test for
>> https://github.com/geotools/geotools/pull/2810 [GEOS-9135] Support WFS-T
>> on tables with unique guid as primary key, or could you assist me, so that
>> we can get the PR into GeoServer?
>>
>> Thanks
>>
>> Peter
>>
>> On Sun, 1 Mar 2020 at 15:57, Jody Garnett <[email protected]> wrote:
>>
>>> Got my calendar wrong!
>>>
>>> I expect to work on the RC on Monday, please advise on any fixes or PRs
>>> that should be included.
>>>
>>> (I have one regression on mbstyle I am working on this weekend)
>>>
>>> On Tue, Feb 18, 2020 at 10:27 AM Jody Garnett <[email protected]>
>>> wrote:
>>>
>>>> Reminder release candidate is scheduled for March 27th, please help
>>>> review pull-requests and support those pushing towards the finish line.
>>>> --
>>>> Jody Garnett
>>>>
>>> --
>>> --
>>> Jody Garnett
>>> _______________________________________________
>>> Geoserver-devel mailing list
>>> [email protected]
>>> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>>>
>>
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to