GeoServer
  1. GeoServer
  2. GEOS-3549

wcs 1.0 module with multi dimensional coverage support

    Details

    • Type: Task Task
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.3.5, 2.4-beta
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      2
    1. GeoServer-2.0-WCS100-EMF-rev13396.diff
      523 kB
      Justin Deoliveira
    2. GeoServer-trunk-WCS10EMF-rev13423.diff
      981 kB
      Alessio Fabiani

      Activity

      Hide
      Justin Deoliveira added a comment -

      Looked at the patch briefly. First off it seems like there are a few changes that are non necessary. FOr instance the changes to src/wcs/src/main/java/applicationContext.xml don't actually seem to change anything unless i am missing something. There are also quite a bit of trivial formatting changes which makes it hard to track what is actually changing.

      However for the most part the patch seems to add a new module wcs1_0? What about the existing wcs module? Is this intended to be a replacement?

      Show
      Justin Deoliveira added a comment - Looked at the patch briefly. First off it seems like there are a few changes that are non necessary. FOr instance the changes to src/wcs/src/main/java/applicationContext.xml don't actually seem to change anything unless i am missing something. There are also quite a bit of trivial formatting changes which makes it hard to track what is actually changing. However for the most part the patch seems to add a new module wcs1_0? What about the existing wcs module? Is this intended to be a replacement?
      Hide
      Alessio Fabiani added a comment -

      The patch I sent you was not the final one, the idea is to create a new module WCS1_0 very similar to the WCS1_1 which will take care of all the WCS 1.0.0 protocol operations. The old WCS module will contain only the common parts and utilities shared between WCS1_0 and WCS1_1 modules, but this is a thing we can discuss further if you think that there is no reason to maintian another WCS module on the source. Maybe we can move the utility classes somewhere else.

      Show
      Alessio Fabiani added a comment - The patch I sent you was not the final one, the idea is to create a new module WCS1_0 very similar to the WCS1_1 which will take care of all the WCS 1.0.0 protocol operations. The old WCS module will contain only the common parts and utilities shared between WCS1_0 and WCS1_1 modules, but this is a thing we can discuss further if you think that there is no reason to maintian another WCS module on the source. Maybe we can move the utility classes somewhere else.
      Hide
      Andrea Aime added a comment -

      I would be ok with both approaches in the long term, one that puts all the code in a single module, and one that has three, wcs/core wcs/wcs1_0, wcs/wcs1_1

      I surely hope the differences between the two modules end up being the xml and kvp parsers, and the response encoders, but that the raster processing logic is fully shared.

      Show
      Andrea Aime added a comment - I would be ok with both approaches in the long term, one that puts all the code in a single module, and one that has three, wcs/core wcs/wcs1_0, wcs/wcs1_1 I surely hope the differences between the two modules end up being the xml and kvp parsers, and the response encoders, but that the raster processing logic is fully shared.
      Hide
      Alessio Fabiani added a comment -

      I'm attaching to the JIRA the new updated and definitive patch for WCS1.0 EMF bindings ... it requires first the patch on GeoTools as for JIRA GEOT-2738

      Show
      Alessio Fabiani added a comment - I'm attaching to the JIRA the new updated and definitive patch for WCS1.0 EMF bindings ... it requires first the patch on GeoTools as for JIRA GEOT-2738
      Hide
      Simone Giannecchini added a comment -

      It's been in since ages. Closing.

      Show
      Simone Giannecchini added a comment - It's been in since ages. Closing.

        People

        • Assignee:
          Alessio Fabiani
          Reporter:
          Justin Deoliveira
        • Votes:
          0 Vote for this issue
          Watchers:
          3 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved: