Doesn't protobuf ensure forwards compatibility? Why would it break?
At worse, you can include the changes necessary for it to compile
cleanly, without adding support for the new fields/methods?
Le 22/06/2023 à 02:16, Sutou Kouhei a écrit :
Hi,
The following part in the original e-mail is the one:
https://github.com/apache/arrow/pull/36009 is an
implementation of this proposal. The pull requests has the
followings:
1. Format changes:
* format/Flight.proto
https://github.com/apache/arrow/pull/36009/files#diff-53b6c132dcc789483c879f667a1c675792b77aae9a056b257d6b20287bb09dba
* format/FlightSql.proto
https://github.com/apache/arrow/pull/36009/files#diff-fd4e5266a841a2b4196aadca76a4563b6770c91d400ee53b6235b96da628a01e
2. Documentation changes:
docs/source/format/Flight.rst
https://github.com/apache/arrow/pull/36009/files#diff-839518fb41e923de682e8587f0b6fdb00eb8f3361d360c2f7249284a136a7d89
We can split the part to a separated pull request. But if we
split the part and merge the pull requests for format
related changes and implementation related changes
separately, our CI will be broken temporary. Because our
implementations use auto-generated sources that are based on
*.proto.
Thanks,