Skip to content

Conversation

@Fokko
Copy link
Contributor

@Fokko Fokko commented Oct 19, 2023

I think we should remove the excluding weekends part of the wait period. With a patch release like we're doing now, I don't think we want to be constrained by this. In practice, I don't think many releases are being published over the weekend.

Apache states that a vote should be open for 72 hours. https://www.apache.org/foundation/voting.html#expressing-votes-1-0-1-and-fractions.

If we agree on it, I'll open up a PR on the docs site as well 👍

I think we should remove the excluding weekends part
of the wait period. With a patch release like we're doing now, I don't think we want to be constrained by this. In practice, I don't think many releases are being published over the weekend.

Apache states that a vote **should** be open for 72 hours.
https://www.apache.org/foundation/voting.html#expressing-votes-1-0-1-and-fractions.

If we agree on it, I'll open up a PR on the docs site as well 👍
@github-actions github-actions bot added the INFRA label Oct 19, 2023
@Fokko
Copy link
Contributor Author

Fokko commented Oct 19, 2023

Some historical context: apache/iceberg-docs#187 (comment)

Copy link
Member

@hussein-awala hussein-awala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, I don't think the other ASF projects exclude the weekends from the voting period (at least Airflow, Spark, Kafka, and Hudi don't do that).

https://www.apache.org/legal/release-policy.html#release-approval

Copy link
Member

@jbonofre jbonofre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, and from Apache standpoint, weekend doesn't matter :) I will create another PR about updating DOAP file post release.

@nastra nastra merged commit 4e05dcf into main Oct 25, 2023
@nastra nastra deleted the fd-remove-weekends branch October 25, 2023 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants