Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

javascript and int64 as map key #21

Open
Notmeor opened this issue Oct 14, 2020 · 5 comments
Open

javascript and int64 as map key #21

Notmeor opened this issue Oct 14, 2020 · 5 comments
Labels
bug Something isn't working javascript triaged Issue has been triaged

Comments

@Notmeor
Copy link

Notmeor commented Oct 14, 2020

What language does this apply to?
javascript

Describe the problem you are trying to solve.
With an int64 message field, one could annotate it with the option [jstype=JS_STRING], but what to do with an int64 map key?

Describe the solution you'd like
Support for [jstype=JS_STRING] option with map key/value(or it's already there, but with different syntax?)

Describe alternatives you've considered

Additional context
I'm using protoc 3.6.1, with proto2 syntax

@perezd
Copy link

perezd commented Oct 22, 2020

We're internally discussing the concept of MapOptions that would be used to facilitate this. I'll keep you posted!

@perezd perezd self-assigned this Oct 22, 2020
@Notmeor
Copy link
Author

Notmeor commented Oct 29, 2020

We're internally discussing the concept of MapOptions that would be used to facilitate this. I'll keep you posted!

looking forward to it!

@EchoZhaoH
Copy link

Is there any solution?

@acozzette acozzette transferred this issue from protocolbuffers/protobuf May 16, 2022
@dibenede dibenede added bug Something isn't working triaged Issue has been triaged labels Sep 2, 2022
@dibenede
Copy link
Contributor

dibenede commented Sep 2, 2022

This is a limitation of the current API. We're thinking of introducing an abstraction for 64 bit integers to deal with this.

@dolav
Copy link

dolav commented Dec 17, 2024

Is there any update on this topic?
What is the best way to define such a map with the existing support?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working javascript triaged Issue has been triaged
Projects
None yet
Development

No branches or pull requests

5 participants