Hi guys,
all the stuff for the WCS 1.0.0 EMF reviews has been done and
committed and replies have been added to the comments. If you would
like to take a look at this, otherwise I guess we can close the
reviews and move on.
Cheers,
Alessio.
--
-------------------------------------------------------
Eng. Alessio Fabiani
Vice-President /CTO GeoSolutions S.A.S.
Via Carignoni 51
55041 Camaiore (LU)
Italy
phone: +39 0584983027
fax: +39 0584983027
mob: +39 349 8227000
http://www.geo-solutions.it
-------------------------------------------------------
Alessio Fabiani ha scritto:
Hi guys,
all the stuff for the WCS 1.0.0 EMF reviews has been done and
committed and replies have been added to the comments. If you would
like to take a look at this, otherwise I guess we can close the
reviews and move on.
I don't remember having any other concerns. If everything has been
addressed, I believe we can move to the next stage.
Is the improved module passing the CITE tests, passing its own tests
(you added some, right?) and working ok for you?
If so, I personally have no issues in merging it to 1.7.x.
Arguably, it's quite a change, we should wait for 1.7.0 to be
released before mering, but given the very low amount of users actually hitting WCS (no real user request in the last year that I can remember besides "what is WCS anyways?") I'm not feeling like blocking your work
would do us any good.
Anyways, this would be better discussed with the PSC, you have my
+1 on merging. Let's hear what the others have to say.
Cheers
Andrea
Andrea Aime ha scritto:
Arguably, it's quite a change, we should wait for 1.7.0 to be
released before mering, but given the very low amount of users actually hitting WCS (no real user request in the last year that I can remember besides "what is WCS anyways?") I'm not feeling like blocking your work
would do us any good.
Anyways, this would be better discussed with the PSC, you have my
+1 on merging. Let's hear what the others have to say.
Oh, forgot one crucial condition for me being +1 to the merge: it has
to touch only the wcs modules. Any change that might be required to main/wms/wfs/whatever will have to be reviewed very attentively
(I'd really prefer to see none actually).
Cheers
Andrea
Hi Andrea, all,
the changes do not affect other geoserver modules at all, also WCS 1.1
is untouched.
The merged code is ready to be committed. Once the other guys give me
the OK I will commit it.
Thanks,
Alessio.
On Thu, Aug 21, 2008 at 12:08 PM, Andrea Aime <aaime@anonymised.com> wrote:
Andrea Aime ha scritto:
Arguably, it's quite a change, we should wait for 1.7.0 to be
released before mering, but given the very low amount of users actually
hitting WCS (no real user request in the last year that I can remember
besides "what is WCS anyways?") I'm not feeling like blocking your work
would do us any good.
Anyways, this would be better discussed with the PSC, you have my
+1 on merging. Let's hear what the others have to say.
Oh, forgot one crucial condition for me being +1 to the merge: it has
to touch only the wcs modules. Any change that might be required to
main/wms/wfs/whatever will have to be reviewed very attentively
(I'd really prefer to see none actually).
Cheers
Andrea
--
-------------------------------------------------------
Eng. Alessio Fabiani
Vice-President /CTO GeoSolutions S.A.S.
Via Carignoni 51
55041 Camaiore (LU)
Italy
phone: +39 0584983027
fax: +39 0584983027
mob: +39 349 8227000
http://www.geo-solutions.it
-------------------------------------------------------
It sounds like it'd be ok with putting it in, but we are in RC status, which is supposed to be as much of a freeze as we ever do. I think if the next release we're doing is 1.7.0 it'd be better to wait to commit till right after it comes out, hopefully early next week. If we're doing RC2 next then I could see it being ok to put this in for that release.
C
Alessio Fabiani wrote:
Hi Andrea, all,
the changes do not affect other geoserver modules at all, also WCS 1.1
is untouched.
The merged code is ready to be committed. Once the other guys give me
the OK I will commit it.
Thanks,
Alessio.
On Thu, Aug 21, 2008 at 12:08 PM, Andrea Aime <aaime@anonymised.com> wrote:
Andrea Aime ha scritto:
Arguably, it's quite a change, we should wait for 1.7.0 to be
released before mering, but given the very low amount of users actually
hitting WCS (no real user request in the last year that I can remember
besides "what is WCS anyways?") I'm not feeling like blocking your work
would do us any good.
Anyways, this would be better discussed with the PSC, you have my
+1 on merging. Let's hear what the others have to say.
Oh, forgot one crucial condition for me being +1 to the merge: it has
to touch only the wcs modules. Any change that might be required to
main/wms/wfs/whatever will have to be reviewed very attentively
(I'd really prefer to see none actually).
Cheers
Andrea
I have to agree with Chris here... I know the new module is better, much higher quality, passes cite, and the old module is more or less unused but I would still be against making such a major change when we are in RC stage.
The only thing I can recommend doing in the interim is throwing it in community, and giving you a problem which turns it on and the old module off. However that might be more work than just waiting for 1.7.0 to go out.
-Justin
Chris Holmes wrote:
It sounds like it'd be ok with putting it in, but we are in RC status, which is supposed to be as much of a freeze as we ever do. I think if the next release we're doing is 1.7.0 it'd be better to wait to commit till right after it comes out, hopefully early next week. If we're doing RC2 next then I could see it being ok to put this in for that release.
C
Alessio Fabiani wrote:
Hi Andrea, all,
the changes do not affect other geoserver modules at all, also WCS 1.1
is untouched.
The merged code is ready to be committed. Once the other guys give me
the OK I will commit it.
Thanks,
Alessio.
On Thu, Aug 21, 2008 at 12:08 PM, Andrea Aime <aaime@anonymised.com> wrote:
Andrea Aime ha scritto:
Arguably, it's quite a change, we should wait for 1.7.0 to be
released before mering, but given the very low amount of users actually
hitting WCS (no real user request in the last year that I can remember
besides "what is WCS anyways?") I'm not feeling like blocking your work
would do us any good.
Anyways, this would be better discussed with the PSC, you have my
+1 on merging. Let's hear what the others have to say.
Oh, forgot one crucial condition for me being +1 to the merge: it has
to touch only the wcs modules. Any change that might be required to
main/wms/wfs/whatever will have to be reviewed very attentively
(I'd really prefer to see none actually).
Cheers
Andrea
--
Justin Deoliveira
Software Engineer, OpenGeo
http://opengeo.org