SandishKumarHN opened a new pull request, #10030:
URL: https://github.com/apache/pinot/pull/10030
pinot-protobuf plugin changes:
- This pull request introduces a new `feature` that allows users to specify
the protobuf message java class name as a mandatory field, replacing the
descriptorFile field. along with the existing option of protobuf message name
and `.desc` file. it will be easier for users to pass a class name than
generate descriptor files.
- Added new build descriptor methods to generate a descriptor using either
the full qualified protobuf class name or a .desc file and message name.
`buildDescriptor(String messageName, String descFilePathOpt)` and
`buildDescriptorFromJavaClass(messageName)`
- Additionally, this PR removes the dependency on the third-party library
`com.github.os72` for generating protobuf descriptor and renames all instances
of `ProtoBuf*` to `Protobuf*`.
cc: @npawar @KKcorps
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
