Re: [I] feat: in standalone mode (without etcd), protobuf with import is not loaded [apisix]

["davendu (via GitHub)" <gi...@apache.org>]

davendu commented on issue #11486:
URL: https://github.com/apache/apisix/issues/11486#issuecomment-2936482807

   > Multiple proto files can be merged temporarily using an external tool. Waiting for others to contribute this feature.
   
   @Baoyuantop I'm not quite sure if I understand your idea well. Do you mean there should be a tool, which works like C/C++ preprocessor, and merge a folder of `*.proto` into single one?
   
   If so, that would be a creative idea, but I doubt if it works, due to these reasons:
   
   1. Namespace collision. At least in my project, protobuf files resides in different folder is mean for different package. And there are some name collisions.
   2. Version conflicts. Protobuf have two versions `proto2` and `proto3. According to [the document](https://protobuf.dev/programming-guides/proto3/#proto2), you can import `proto2` file within a `proto3` one, and vice versa. Merging files together would have weird glitches, I think.
   
   I strongly advice we find a proper Protobuf parsing library and use it.


-- 
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: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org