[postgis-tickets] [PostGIS] #4128: ST_AsMVT: add feature id support

[PostGIS]

#4128: ST_AsMVT: add feature id support
---------------------------+---------------------------
  Reporter:  stepankuzmin  |      Owner:  pramsey
      Type:  enhancement   |     Status:  reopened
  Priority:  medium        |  Milestone:  PostGIS 3.0.0
 Component:  postgis       |    Version:
Resolution:                |   Keywords:
---------------------------+---------------------------

Comment (by Algunenano):

 My proposed changes:
 - Obviously fix the bug casting the Datum to the appropriate type.
 - Match the geometry column behaviour: Do the validation once, including
 type, and fail if not found.
 - Do not include the id column as a property too. If you want the
 information duplicated you can add the column twice. Document this.
 - Silently ignore (leave them as id not set) NULLs and negative values
 (not supported by MVT). If you want a default value you can set it up
 before passing the information to the aggregation function. Document this.
 - The column is chosen as the first column with the correct name and valid
 types (smallint, integer, bigint). Properties inside JSON values are not
 supported. Document this.
 - Add tests around all these behaviours.

 PR coming.

-- 
Ticket URL: <https://trac.osgeo.org/postgis/ticket/4128#comment:11>
PostGIS <http://trac.osgeo.org/postgis/>
The PostGIS Trac is used for bug, enhancement & task tracking, a user and developer wiki, and a view into the subversion code repository of PostGIS project.