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

Incorporate TCompact Protocol #10

Open
anthonyatto opened this issue May 15, 2019 · 1 comment
Open

Incorporate TCompact Protocol #10

anthonyatto opened this issue May 15, 2019 · 1 comment

Comments

@anthonyatto
Copy link

I read in another issue related to implyr that this package would only be developed to fit @marekjagielski needs and will not be actively managed moving forward.

I would guess that @ianmcook and I are interested in this package for the same reason: parsing impala query log files. The only thread I found regarding this matter is found here and it seems as like using the TCompactProtocol is the solution.

Given, I wanted to ask if there are any plans to include the TCompactProtocol?

@marekjagielski
Copy link
Member

Hi,
If I am going to implement TCompactProtocol, this would not happen this year for sure. Anyway, it would be nice to have this feature. It seems that development shouldn't be too difficult. Please look at python version: https://github.com/Thriftpy/thriftpy2/blob/0e9608798787af8b62b57299e43b0a09e9db0c09/thriftpy2/protocol/compact.py
Here you have the R implementation of binary protocol:
https://github.com/systemincloud/thriftr/blob/master/R/protocol_binary.R
Contributions are welcome.

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

2 participants