locationtech / geowave

GeoWave provides geospatial and temporal indexing on top of Accumulo, HBase, BigTable, Cassandra, Kudu, Redis, RocksDB, and DynamoDB.
Apache License 2.0
502 stars 190 forks source link

GeoServer add layer fails with no error #1645

Closed pfrollo closed 4 years ago

pfrollo commented 4 years ago

When I add a layer to GeoServer and no layer is added, there is no error given by CLI indicating that no layer was added.

I expect when adding a layer fails that a clear message is given by CLI indicating that no layer was added.

I followed this tutorial and when executed geowave gs layer add gdelt-kde I got the following message:

Add GeoServer layer for 'gdelt-kde: OK : {
  "description": "Please use -a, or choose one of these with -id:",
  "layers":   [
        {
      "id": "gdeltevent_kde",
      "type": "raster"
    },
        {
      "id": "gdeltevent",
      "type": "vector"
    }
  ]
}

This message is generated by org.locationtech.geowave.cli.geoserver.GeoServerRestClient#addLayer. Further inspection of the code reveals that no layer was added in this case because addlayer doesn't know how to deal with multiple adapters.

pfrollo commented 4 years ago

Addressed in PR: https://github.com/locationtech/geowave/pull/1647