Hi,
I was chatting with Jody on IRC and it seems the feature collection work on
the Geotools side is done with a pull request ready to be merged.
uDig seems to be ready as well, whilst GeoServer wise as far as I understand
we have a branch that Justin made that also works, but no pull request.
If everything is ready can we also have the pull request on the Geoserver side
so that we can give it a kick and call this refactor done?
Sounds good on this end, will make a 9.0-M0 tag before the patch is applied (so there is a safe point to latch onto for any projects that need it).
–
Jody Garnett
On Saturday, 10 November 2012 at 7:09 PM, Andrea Aime wrote:
Hi,
I was chatting with Jody on IRC and it seems the feature collection work on
the Geotools side is done with a pull request ready to be merged.
uDig seems to be ready as well, whilst GeoServer wise as far as I understand
we have a branch that Justin made that also works, but no pull request.
If everything is ready can we also have the pull request on the Geoserver side
so that we can give it a kick and call this refactor done?
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_nov
Hi,
I was chatting with Jody on IRC and it seems the feature collection work on
the Geotools side is done with a pull request ready to be merged.
uDig seems to be ready as well, whilst GeoServer wise as far as I understand
we have a branch that Justin made that also works, but no pull request.
I pushed the changes up to my own repo last weekend and was waiting for a review, specially of the secure feature collection part of the changes. Perhaps I missed it but never heard back if it looked ok or not.
If everything is ready can we also have the pull request on the Geoserver side
so that we can give it a kick and call this refactor done?
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_nov
The changes were smoothly applied, did not see any major workarounds like in GeoTools or uDig
2 We could make use of some of those DataUtilities methods.
I tend to do try / finally around each FeatureIterator
The security feature collections were fine, I wanted to make sure there was no way to accidentally get to the delegate and “add” anything. And yes they are all wrapped up
–
Jody Garnett
On Monday, 12 November 2012 at 10:46 AM, Justin Deoliveira wrote:
Hi,
I was chatting with Jody on IRC and it seems the feature collection work on
the Geotools side is done with a pull request ready to be merged.
uDig seems to be ready as well, whilst GeoServer wise as far as I understand
we have a branch that Justin made that also works, but no pull request.
I pushed the changes up to my own repo last weekend and was waiting for a review, specially of the secure feature collection part of the changes. Perhaps I missed it but never heard back if it looked ok or not.
If everything is ready can we also have the pull request on the Geoserver side
so that we can give it a kick and call this refactor done?
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_nov
–
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_nov
Hi,
I was chatting with Jody on IRC and it seems the feature collection work on
the Geotools side is done with a pull request ready to be merged.
uDig seems to be ready as well, whilst GeoServer wise as far as I understand
we have a branch that Justin made that also works, but no pull request.
I pushed the changes up to my own repo last weekend and was waiting for a review, specially of the secure feature collection part of the changes. Perhaps I missed it but never heard back if it looked ok or not.
Ops, sorry Justin, I definitely reviewed the changes and they were ok, believe I’ve wrote
something down too on the pull request but I guess I got distracted and failed to
send the message… sorry!
Hi,
I was chatting with Jody on IRC and it seems the feature collection work on
the Geotools side is done with a pull request ready to be merged.
uDig seems to be ready as well, whilst GeoServer wise as far as I understand
we have a branch that Justin made that also works, but no pull request.
I pushed the changes up to my own repo last weekend and was waiting for a review, specially of the secure feature collection part of the changes. Perhaps I missed it but never heard back if it looked ok or not.
Ops, sorry Justin, I definitely reviewed the changes and they were ok, believe I’ve wrote
something down too on the pull request but I guess I got distracted and failed to
send the message… sorry!
Yep. We are ready now ( after your branch is rebased ), it chose to avoid the work week.
Are you okay for Friday afternoon Justin? I can join you on IRC and push the button. then it should be done by the time Andrea wakes up and checks what happened to rendering performance.
Hi,
I was chatting with Jody on IRC and it seems the feature collection work on
the Geotools side is done with a pull request ready to be merged.
uDig seems to be ready as well, whilst GeoServer wise as far as I understand
we have a branch that Justin made that also works, but no pull request.
I pushed the changes up to my own repo last weekend and was waiting for a review, specially of the secure feature collection part of the changes. Perhaps I missed it but never heard back if it looked ok or not.
Ops, sorry Justin, I definitely reviewed the changes and they were ok, believe I’ve wrote
something down too on the pull request but I guess I got distracted and failed to
send the message… sorry!
–
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_nov
Yep. We are ready now ( after your branch is rebased ), it chose to avoid the work week.
Are you okay for Friday afternoon Justin? I can join you on IRC and push the button. then it should be done by the time Andrea wakes up and checks what happened to rendering performance.
Yup, sure i will be around. And in the meantime I will try to merge in the latest to my branch (i never rebase a public branch until doing a merge back into a primary branch) .
Hi,
I was chatting with Jody on IRC and it seems the feature collection work on
the Geotools side is done with a pull request ready to be merged.
uDig seems to be ready as well, whilst GeoServer wise as far as I understand
we have a branch that Justin made that also works, but no pull request.
I pushed the changes up to my own repo last weekend and was waiting for a review, specially of the secure feature collection part of the changes. Perhaps I missed it but never heard back if it looked ok or not.
Ops, sorry Justin, I definitely reviewed the changes and they were ok, believe I’ve wrote
something down too on the pull request but I guess I got distracted and failed to
send the message… sorry!
–
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_nov