kbendick commented on code in PR #109:
URL: https://github.com/apache/iceberg-docs/pull/109#discussion_r915464688


##########
landing-page/content/common/roadmap.md:
##########
@@ -30,33 +30,18 @@ Related design docs will be linked on the planning boards.
 # Priority 1
 
 * API: [Iceberg 1.0.0](https://github.com/apache/iceberg/projects/3) [medium]
-* Spark: [Merge-on-read plans](https://github.com/apache/iceberg/projects/11) 
[large]
-* Maintenance: [Delete file 
compaction](https://github.com/apache/iceberg/projects/10) [medium]
-* Flink: [Upgrade to 1.13.2](https://github.com/apache/iceberg/projects/12) 
(document compatibility) [medium]
 * Python: [Pythonic refactor](https://github.com/apache/iceberg/projects/7) 
[medium]
+* Spec: [Z-ordering / Space-filling 
curves](https://github.com/apache/iceberg/projects/16) [medium]
+* Spec: [Snapshot tagging and 
branching](https://github.com/apache/iceberg/projects/4) [small]
+* Views: [Spec](https://github.com/apache/iceberg/projects/6) [medium]

Review Comment:
   I was going to suggest adding the word “additional” to not confuse people 
less familiar with the project - as statistics are somewhat integral to the 
Iceberg spec already - something like `Implement additional statistics 
information in table snapshot`.
   
   But I see it came directly from the PR title so I think it’s fine. 
Eventually we should consult with Piotr for a more optimal high level 
description of the milestone but this is a good stand in for now (and possibly 
even for the medium term).



##########
landing-page/content/common/roadmap.md:
##########
@@ -30,33 +30,20 @@ Related design docs will be linked on the planning boards.
 # Priority 1
 
 * API: [Iceberg 1.0.0](https://github.com/apache/iceberg/projects/3) [medium]
-* Spark: [Merge-on-read plans](https://github.com/apache/iceberg/projects/11) 
[large]
-* Maintenance: [Delete file 
compaction](https://github.com/apache/iceberg/projects/10) [medium]
-* Flink: [Upgrade to 1.13.2](https://github.com/apache/iceberg/projects/12) 
(document compatibility) [medium]
 * Python: [Pythonic refactor](https://github.com/apache/iceberg/projects/7) 
[medium]
+* Spec: [Z-ordering / Space-filling 
curves](https://github.com/apache/iceberg/projects/16) [medium]
+* Spec: [Snapshot tagging and 
branching](https://github.com/apache/iceberg/projects/4) [small]
+* Views: [Spec](https://github.com/apache/iceberg/projects/6) [medium]
+* Puffin: [Implement statistics information in table 
snapshot](https://github.com/apache/iceberg/pull/4741) [medium]
+* Flink: [FLIP-27 based Iceberg 
source](https://github.com/apache/iceberg/projects/23) [large]
 
 # Priority 2
 
 * ORC: [Support delete files stored as 
ORC](https://github.com/apache/iceberg/projects/13) [small]
 * Spark: [DSv2 streaming 
improvements](https://github.com/apache/iceberg/projects/2) [small]
 * Flink: [Inline file 
compaction](https://github.com/apache/iceberg/projects/14) [small]
 * Flink: [Support UPSERT](https://github.com/apache/iceberg/projects/15) 
[small]
-* Flink: [FLIP-27 based Iceberg 
source](https://github.com/apache/iceberg/projects/23) [large]
-* Views: [Spec](https://github.com/apache/iceberg/projects/6) [medium]
-* Spec: [Z-ordering / Space-filling 
curves](https://github.com/apache/iceberg/projects/16) [medium]

Review Comment:
   For z-ordering, I think there’s still some work left to be done - though the 
MVP is complete for sure.
   
   EDIT - I just realized this says `Spec`.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to