aiokafka icon indicating copy to clipboard operation
aiokafka copied to clipboard

Create own partitioner

Open pawlyk opened this issue 6 years ago • 2 comments

Port murmur2 from python-kafka and rewrite in cython for better performance.

pawlyk avatar Sep 15 '19 16:09 pawlyk

Codecov Report

Merging #554 into master will decrease coverage by 73.51%. The diff coverage is 50%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #554       +/-   ##
===========================================
- Coverage   98.42%   24.91%   -73.52%     
===========================================
  Files          31       34        +3     
  Lines        5017     5058       +41     
===========================================
- Hits         4938     1260     -3678     
- Misses         79     3798     +3719
Impacted Files Coverage Δ
aiokafka/producer/producer.py 22.44% <100%> (-77.05%) :arrow_down:
aiokafka/partitioner/__init__.py 100% <100%> (ø)
aiokafka/partitioner/default.py 38.46% <38.46%> (ø)
aiokafka/partitioner/hashed.py 43.47% <43.47%> (ø)
aiokafka/consumer/group_coordinator.py 9.51% <0%> (-89.66%) :arrow_down:
aiokafka/consumer/fetcher.py 10.55% <0%> (-86.3%) :arrow_down:
aiokafka/producer/sender.py 14.54% <0%> (-85%) :arrow_down:
aiokafka/record/util.py 14.13% <0%> (-84.79%) :arrow_down:
aiokafka/cluster.py 16.21% <0%> (-83.79%) :arrow_down:
aiokafka/client.py 14.64% <0%> (-83.44%) :arrow_down:
... and 17 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update c0fa11f...e76f66f. Read the comment docs.

codecov[bot] avatar Sep 16 '19 08:09 codecov[bot]

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

:x: Pavel Stadnikov
:x: pawlyk


Pavel Stadnikov seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Nov 16 '20 10:11 CLAassistant