-
Notifications
You must be signed in to change notification settings - Fork 140
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
Bump msgpack version #171
Bump msgpack version #171
Conversation
Signed-off-by: Nilesh Patra <npatra974@gmail.com>
71c3df5
to
5b798ff
Compare
1 similar comment
@arcivanov Do you have any concern for this patch? |
Yes, I do @repeatedly. Specifically due to #157. #157 provides a description and logs of a specific in-FluentD unpacking failure requiring msgpack pinning pre-1.0. The Fluent Logger 1.0 has a harness rewrite that would allow compatibility testing with whatever versions of fluentd/-bit but I simply don't have time to work on it right now. |
@@ -68,7 +68,7 @@ def get_received(self): | |||
self._buf.seek(0) | |||
# TODO: have to process string encoding properly. currently we assume | |||
# that all encoding is utf-8. | |||
return list(Unpacker(self._buf, encoding='utf-8')) | |||
return list(Unpacker(self._buf)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing the mock server to "make tests pass" does nothing to alleviate concerns in #157
We are starting to get unresolvable venvs because of the upper-pin. Is there anything we can do to help here? |
I'll remove the pin today in that case. |
fixes fluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixes fluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixes fluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixes fluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixes fluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz> Signed-off-by: Matt Cipperly <mcipperly@fb.com>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixes fluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixes fluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Remove Python 2.7 to remove the msgpack encoding ambiguity issue. Remove Python 3.4 support just because it's dead. fixes fluent#171 Signed-off-by: Arcadiy Ivanov <arcadiy@ivanov.biz>
Since the
encoding
option is always UTF-8 with msgpack>=1, I've remove the same.Otherwise it fails with:
After the change, it goes green: