substrait-io / substrait

A cross platform way to express data transformation, relational algebra, standardized record expression and plans.
https://substrait.io
Apache License 2.0
1.14k stars 148 forks source link

docs: specify Substrait library merge policy #603

Closed vbarua closed 5 months ago

vbarua commented 6 months ago

Making the current library merge policy more explicit.

vbarua commented 5 months ago

@westonpace

This effectively means that each Substrait library needs two committers to make progress right?

Yes, assuming that the contributor is a committer. Only one commiter is needed for external contributions.

This is effectively the implicit policy for substrait-java.

That seems like it could be a barrier for new / less popular libraries.

That is definitely a tension with this change. I would be open open to loosening the policy in some cases, I'm just not sure how to word it. I'm aiming to codify our implicit policies in writing.