So going back to this proposal of last year. I contributed a first batch
of code which is inside a new PR. Now this is a new community module
that will be fully maintained by me (and fully documented soon as well)
so I do not think I technically need approval for the code (although
reviews/comments are always more then welcome!) but there are two small
changes in the "main module" that I do need approval for. Would there
someone will so kind to have a look at them (would only take a few minutes).
These are direct links to the changes in main:
https://github.com/geoserver/geoserver/pull/2759/commits/6945957a93b326ddaa39ee00c3ca4a707d663aee#diff-ed7269d9c2be90db1cbfde7d0f32d751
https://github.com/geoserver/geoserver/pull/2759/commits/6945957a93b326ddaa39ee00c3ca4a707d663aee#diff-5fac1624412cebb8475c6e0fee6fd17c
Kind Regards
Niels
On 30-05-17 16:20, Niels Charlier wrote:
What is your opinion on the dependency concern?
What do you think of having the new module be called "task manager" or
something like that.
Although it will contain a quite a bit of task implementation that
regards remote data management specifically. Should I add two new
modules though :/
Regards
Niels
On 30-05-17 04:05, Ben Caradoc-Davies wrote:
On 30/05/17 00:55, Niels Charlier wrote:
Therefore, logically, it would be importer that needs be modified to be
built on top of my API rather than me building on top of importer.
This sounds like a great option. A modest investment in refactoring
now will save us the future maintenance overhead introduced by
duplication. (And I speak us someone currently fixing my own
ill-considered DataUtilities duplication from five years ago.)
Kind regards,
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
