You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "szehon-ho (via GitHub)" <gi...@apache.org> on 2026/03/26 20:21:52 UTC
[PR] Spec: Change wording of 'custom' to 'non-defualt' for Geo type crs [iceberg]
szehon-ho opened a new pull request, #15781:
URL: https://github.com/apache/iceberg/pull/15781
The wording 'custom' causes some confusion. It be taken to mean a 'custom' CRS not defined in existing authorities. Whereas here, actually these are very common CRS's. I think it's more clear to use the word 'non-default' here, especially as we just mention 'default' just above.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-default' for Geo type crs [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed pull request #15781: Spec: Change wording of 'custom' to 'non-default' for Geo type crs
URL: https://github.com/apache/iceberg/pull/15781
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-default' for Geo type crs [iceberg]
Posted by "rdblue (via GitHub)" <gi...@apache.org>.
rdblue commented on PR #15781:
URL: https://github.com/apache/iceberg/pull/15781#issuecomment-4158199731
> Although the wording is more clear it makes the spec more restrictive.
How does this make it more restrictive? I didn't read it that way.
What can we do to make this more clear? I agree that the cases you listed are reasonable ways to specify a CRS -- how should we change the wording to definitely allow them?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-default' for Geo type crs [iceberg]
Posted by "mkaravel (via GitHub)" <gi...@apache.org>.
mkaravel commented on PR #15781:
URL: https://github.com/apache/iceberg/pull/15781#issuecomment-4298911145
@szehon-ho I think we can close this PR in favor of https://github.com/apache/parquet-format/pull/560
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-default' for Geo type crs [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #15781:
URL: https://github.com/apache/iceberg/pull/15781#issuecomment-4580961293
This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-default' for Geo type crs [iceberg]
Posted by "mkaravel (via GitHub)" <gi...@apache.org>.
mkaravel commented on PR #15781:
URL: https://github.com/apache/iceberg/pull/15781#issuecomment-4165881265
@rdblue Please see answers/comments below.
> > Although the wording is more clear it makes the spec more restrictive.
>
> How does this make it more restrictive? I didn't read it that way.
The existing wording (prior to this PR is):
---
_The default CRS value OGC:CRS84 means that the objects must be stored in longitude, latitude based on the WGS84 datum._
_Custom CRS values can be specified by a string of the format type:identifier, where type is one of the following values:_
_srid: [Spatial reference identifier](https://en.wikipedia.org/wiki/Spatial_reference_system#Identifier), identifier is the SRID itself.
projjson: [PROJJSON](https://proj.org/en/stable/specifications/projjson.html), identifier is the name of a table property where the projjson string is stored._
---
How I read this depends on what "custom" means.
Assume for a minute that "custom" means "non-default", that is everything except "OCG:CRS84". In this case what I read is the following: **We have the default value which is `OCG:CRS84` and everything else is non-default. Everything non-default is required to be in one of two formats: (1) `srid:identifier` (e.g. `srid:0`), or (2) `projjson: <projjson string>`**.
Assume that "custom" does not mean "non-default". The natural question is what "custom" means. Given that there are well known authorities my reflex is to think: **There is the default which is `OCG:CRS84`, there are other well-defined CRS via well known authorities (see content in [https://spatialreference.org/](https://spatialreference.org/)), and there is also the possibility of defining my own personal/custom/proprietary CRS in two ways: (1) `srid:identifier` (e.g. `srid:0`), or (2) `projjson: <projjson string>`. I can of course define everything using the two proposed ways for custom CRSs, but I also have the ability to be between the default and the custom ones**. Unfortunately, there is nothing in the wording that either supports or directly rejects the above interpretation, hence the confusion.
>
> What can we do to make this more clear? I agree that the cases you listed are reasonable ways to specify a CRS -- how should we change the wording to definitely allow them?
I think a wording of the following form would be much cleaner:
1. Avoid talking about custom CRSs.
2. Provide valid ways of defining a CRS that includes the two already mentioned plus the ability to specify a CRS using the `authority:code` approach much like what is done in [https://spatialreference.org/](https://spatialreference.org/). The default CRS value of "OGC:CRS84" falls naturally as one of the valid options.
3. Add specific examples of valid CRS definitions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-default' for Geo type crs [iceberg]
Posted by "szehon-ho (via GitHub)" <gi...@apache.org>.
szehon-ho commented on PR #15781:
URL: https://github.com/apache/iceberg/pull/15781#issuecomment-4166153399
I think that goes a bit beyond what i had in mind for the scope of this pr. maybe we can discuss in another pr.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-default' for Geo type crs [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #15781:
URL: https://github.com/apache/iceberg/pull/15781#issuecomment-4523571123
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-defualt' for Geo type crs [iceberg]
Posted by "szehon-ho (via GitHub)" <gi...@apache.org>.
szehon-ho commented on PR #15781:
URL: https://github.com/apache/iceberg/pull/15781#issuecomment-4137977891
@jiayuasu @paleolimbot @mkaravel @rdblue what do you guys think?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Spec: Change wording of 'custom' to 'non-defualt' for Geo type crs [iceberg]
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot commented on PR #15781:
URL: https://github.com/apache/iceberg/pull/15781#issuecomment-4138039715
This makes sense to me!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org