This repository has been archived by the owner on Feb 8, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 27
PulsarApi.proto #106
Comments
Haven't worked this code in forever, but I think the root cause has to do with the core parsing for messages: something was consuming a token and leaving the parser in a bad state. Can you try PR #107? |
jhugard
added a commit
that referenced
this issue
Aug 13, 2019
Address issues with parser order, including issue #106 and possibly #93. Core parser was order dependent on several productions and would confuse a number of things as message definitions. Corrected by adding back-tracking on every production in message. Also fixed running tests when the project path includes a hash symbol, as in .../F#/...
Merged
ctaggart
pushed a commit
that referenced
this issue
Aug 16, 2019
Address issues with parser order, including issue #106 and possibly #93. Core parser was order dependent on several productions and would confuse a number of things as message definitions. Corrected by adding back-tracking on every production in message. Also fixed running tests when the project path includes a hash symbol, as in .../F#/...
@vilinski The commit above was included in release 0.7.2. Can you verify the change worked, so that this issue can be closed? |
sorry, waited for falanx updates it's deps. Falanx 0.6.0 still doesn't work with actual
But type provider works without errors. #r "nuget: Froto.TypeProvider"
open Froto.TypeProvider
let [<Literal>] PulsarApiProto = __SOURCE_DIRECTORY__ + "/PulsarApi.proto"
type PulsarApi = ProtocolBuffersTypeProvider<PulsarApiProto>
let msg = PulsarApi.Pulsar.Proto.MessageIdData()
msg.Partition <- Some 42 so not a Froto issue anymore. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Getting error with the file PulsarApi.proto:
Looks nothing suspect for me on this place:
The text was updated successfully, but these errors were encountered: