Fokko commented on code in PR #10183:
URL: https://github.com/apache/iceberg/pull/10183#discussion_r1616337365
##########
.github/ISSUE_TEMPLATE/iceberg_proposal.yml:
##########
@@ -47,4 +47,10 @@ body:
- label: REST
- label: Puffin
- label: Encryption
- - label: Other
\ No newline at end of file
+ - label: Other
+ - type: checkboxes
Review Comment:
Thanks for your patience here, I was touching grass.
A proposal starts with a Google doc, for example, the Geo support:
https://docs.google.com/document/d/1iVFbrRNEzZl8tDcZC81GFt01QJkLJsI9E2NBOt21IRI
Once there is consensus, the process of splitting it out into separate
issues starts. I'm doubtful if having this additional checkbox and TODO label
clarifies the process.
In addition to that, we started in Python
(https://github.com/apache/iceberg-python/issues/736) and Rust
(https://github.com/apache/iceberg-rust/issues/348) to have tracking issues as
suggested a while ago on the dev-list, which also seem in conflict with having
a tracking issue per proposal.
If the tracking issue works for the partitioned stats, I'm fully okay with
that, just not sure if we should add this to the template (since it is easy to
add a lot of additional checkboxes here, which might raise the bar of proposing
something at all).
--
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]
