forked from jsocol/pystatsd
-
Notifications
You must be signed in to change notification settings - Fork 0
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
3.3.0.1 #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bug: Fix timer decorator with partial functions (jsocol#85)
Update badges
Leave Python 2.6, 3.2, and 3.3 out. Let's focus on >=3.4 (and we'll still support 2.7 because it's easy).
Drop support for old Python versions
Since early June[1], IPv6 loopback address resolution has been flaky on TravisCI. It seems that it should have been flaky even earlier, or there's something strange going on. For now, skip the test. We can come back to diagnosing once master is (otherwise) passing. This failure first started on Python 2.6, but quickly spread to all versions. It appears to pass occasionally, but not on any consistent version. Lately it seems to fail more or less all the time, though on the latest cron build, it did pass on Pypy[2]. [1]: https://travis-ci.org/jsocol/pystatsd/builds/387167216 [2]: https://travis-ci.org/jsocol/pystatsd/builds/417687774
Skip IPv6 resolution test
StatsClientBase and PipelineBase are not intended to be public APIs, and the syntax for metaclasses introduces some complexity between Python 2 and 3. Since ABCMeta and abc.abstractmethod were only being used to guarantee that internal subclasses implemented these methods correctly, and there's no external behavioral issues on Python 3, where the metaclass has been ignored for years, let's remove the metaclass and rely on NotImplementedError() and the test suite as a backstop. Fixes jsocol#106.
Remove dependency on ABCMeta.
Fix minor nits, spelling and unused imports
Collecting thoughts that have been spread across several GitHub issues and pull requests into one place, a reference to use from now on when the discussion of tags comes up.
Allow passing datetime.timedelta objects directly to StatsClient.timing(), automatically converting to milliseconds.
Add support for timedeltas
New client for handling sending stats through Unix domains sockets.
Introduce UnixSocketStatsClient class. Fixes jsocol#76.
Tweaks a lot of doc references to use better Sphinx tooling. Shouldn't break any existing links, but may get to a point where we don't need quite as many named references.
Update Sphinx directives in docs
Remove refs
The client module just broke 300 lines and had a lot of functionality in it. This breaks the single file up into a package and reduces a decent amount of duplication. Makes a few related changes: - Replaces old Python __future__ imports with new ones - Removes __all__ in favor of fewer imports - Refactors common stream code into StreamClientBase - Renames TCPPipeline to StreamPipeline for consistency (this is not a public class name)
Refactor client module into package
Remove old version classifiers
- Drop support for Python 2.5, 2.6, 3.2, 3.3 (jsocol#108, jsocol#116). - Add UnixSocketStatsClient (jsocol#76, jsocol#112). - Add support for timedeltas in timing() (jsocol#104, jsocol#111). - Fix timer decorator with partial functions (jsocol#85). - Remove ABCMeta metaclass (incompatible with Py3) (jsocol#109). - Refactor client module (jsocol#115). - Various doc updates (jsocol#99, jsocol#102, jsocol#110, jsocol#113, jsocol#114).
Fix regex string escape in tests
typo error in directory name
Fix typo in configure.rst
# Conflicts: # setup.py # statsd/client.py
gregflynn
approved these changes
Feb 26, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.