Skip to content

Commit

Permalink
Require minimum length of 1 for primary_geometry #129 (#153)
Browse files Browse the repository at this point in the history
* Require minimum length of 1 for primary_geometry #129

* Update format-specs/geoparquet.md

Co-authored-by: Chris Holmes <chomie@gmail.com>
  • Loading branch information
m-mohr and cholmes authored Dec 5, 2022
1 parent 533ae11 commit 89036f4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
6 changes: 3 additions & 3 deletions format-specs/geoparquet.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,9 +37,9 @@ All file-level metadata should be included under the "geo" key in the parquet me

| Field Name | Type | Description |
| ------------------ | ------ | -------------------------------------------------------------------- |
| version | string | **REQUIRED** The version of the GeoParquet metadata standard used when writing. |
| primary_column | string | **REQUIRED** The name of the "primary" geometry column. |
| columns | object<key, [Column Metadata](#column-metadata)> | **REQUIRED** Metadata about geometry columns, with each key is the name of a geometry column in the table. |
| version | string | **REQUIRED** The version of the GeoParquet metadata standard used when writing. |
| primary_column | string | **REQUIRED** The name of the "primary" geometry column. |
| columns | object<key, [Column Metadata](#column-metadata)> | **REQUIRED** Metadata about geometry columns. Each key is the name of a geometry column in the table. |

At this level, additional implementation-specific fields (e.g. library name) are allowed, and thus readers should be robust in ignoring those.

Expand Down
1 change: 1 addition & 0 deletions format-specs/schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
},
"primary_column": {
"type": "string",
"minLength": 1,
"description": "The name of the 'primary' geometry column."
},
"columns": {
Expand Down

0 comments on commit 89036f4

Please sign in to comment.