Uploaded image for project: 'GeoServer'
  1. GEOS-8765

Undocumented GDAL 2.3.0 CSV output geometry column name change breaks WPSOgrTest

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Medium
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.12.4, 2.13.2, 2.14-RC
    • Component/s: OGR output format, WPS
    • Labels:
      None
    • Environment:

      debian sid gdal-bin 2.3.0+dfsg-1 amd64

      Description

      On debian sid gdal-bin 2.3.0+dfsg-1 amd64:

      Tests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 6.253 sec <<< FAILURE!
      testOGRCSVOutputExecuteDocument(org.geoserver.wps.ogr.WPSOgrTest)  Time elapsed: 214 sec  <<< FAILURE!
      java.lang.AssertionError
      	at org.junit.Assert.fail(Assert.java:86)
      	at org.junit.Assert.assertTrue(Assert.java:41)
      	at org.junit.Assert.assertTrue(Assert.java:52)
      	at org.geoserver.wps.ogr.WPSOgrTest.testOGRCSVOutputExecuteDocument(WPSOgrTest.java:169)
      
      Failed tests:   testOGRCSVOutputExecuteDocument(org.geoserver.wps.ogr.WPSOgrTest)
      

      Failure started after upgrade from GDAL 2.2.4 to GDAL 2.3.0 on Debian sid when processing a GML file with a geometry property called `geometry`. The output CSV geometry column name changed from WKT to geometry:

      $ ogr2ogr --version
      GDAL 2.3.0, released 2018/05/04
      $ ogr2ogr -f CSV -lco GEOMETRY=AS_WKT feature.csv feature.gml; grep STATE feature.csv 
      geometry,gml_id,STATE_NAME
      

      Comparison with Ubuntu Bionic:

      $ ogr2ogr --version
      GDAL 2.2.3, released 2017/11/20
      $ ogr2ogr -f CSV -lco GEOMETRY=AS_WKT feature.csv feature.gml; grep STATE feature.csv
      WKT,gml_id,STATE_NAME
      

      GEOMETRY=AS_WKT is documented to output a geometry column WKT by default.

        Attachments

          Issue links

            Activity

              People

              • Assignee:
                bencaradocdavies Ben Caradoc-Davies
                Reporter:
                bencaradocdavies Ben Caradoc-Davies
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: