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

tarantool protobuf support #2

Closed
despiegk opened this issue Oct 25, 2017 · 5 comments
Closed

tarantool protobuf support #2

despiegk opened this issue Oct 25, 2017 · 5 comments
Assignees
Milestone

Comments

@despiegk
Copy link
Contributor

  • get protobuf to work in tarantool, test performance & memory usage
  • protobuf 3.0
  • make sure prefab is updated to also build required modules for this
@iwanbk
Copy link
Contributor

iwanbk commented Oct 30, 2017

The only Lua lib i found that support protobuf 3 is https://github.com/jinq0123/LuaPbIntf, and i failed to build it on ubuntu 16.04

The other alternative is bind the C implementation https://github.com/protobuf-c/protobuf-c

@iwanbk
Copy link
Contributor

iwanbk commented Oct 30, 2017

The other alternative is bind the C implementation https://github.com/protobuf-c/protobuf-c

It is not a good path because we will need to add Lua bindings for each generated code.

As for LuaPbInft, there is some progress in build, but still failed.

@iwanbk
Copy link
Contributor

iwanbk commented Oct 31, 2017

As for LuaPbInft, there is some progress in build, but still failed.

I still failed to build it, but managed to tried it using docker images from the author https://hub.docker.com/r/vkill/luapbintf/.
(it was build 5 months ago, if we tried to build it now, it will failed).
You can try using my docker https://hub.docker.com/r/iwanbk/docker-tarantool-protobuf/ , i added tarantool and luajit.

Unfortunately, luapbintf doesn't work on LuaJIT.

So, we have some options here:

  • add luajit support to luapbintf. Still no idea what kind of work we need to do. FYI it has C++ components which might complicate the works

  • add partial protobuf 3 support to https://github.com/Neopallium/lua-pb, only features we need.

  • let me know the details why we can't use v2, we might have solution for this.

I think we could try to use last option while we are working on the first option

@despiegk
Copy link
Contributor Author

i'll try capnp in Tarantool, I know we did a test with that one once

@iwanbk
Copy link
Contributor

iwanbk commented Oct 31, 2017

i'll try capnp in Tarantool, I know we did a test with that one once

yep, https://github.com/Incubaid/playenv/tree/master/capnproto/tarantool

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants