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

Packaging improvements / More producer control #10

Merged
merged 18 commits into from Dec 31, 2017

Conversation

borgstrom
Copy link
Contributor

@borgstrom borgstrom commented Dec 29, 2017

  • Improve setup.py by adding a long description, homepage url and other metadata

  • Implement explicit partition & hash keys for KinesisProducer.put -- Fixes Ability to specify partition key #5

  • Allow for explicit max_count & max_size kwargs when creating a KinesisProducer

  • Calculate object size better -- Originally we were just using sys.getsizeof, which only calculated the size of the dictionary structure, not the contents within it (🤦‍♂️ ).

  • Write some unit tests -- more need to be written, but it's a start...

@borgstrom borgstrom changed the title Enable travis for kinesis-python Project improvements Dec 29, 2017
@borgstrom borgstrom closed this Dec 29, 2017
@borgstrom borgstrom reopened this Dec 29, 2017
@codecov-io
Copy link

codecov-io commented Dec 30, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@9cc9dec). Click here to learn what that means.
The diff coverage is 26.47%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #10   +/-   ##
=========================================
  Coverage          ?   28.91%           
=========================================
  Files             ?        4           
  Lines             ?      287           
  Branches          ?        0           
=========================================
  Hits              ?       83           
  Misses            ?      204           
  Partials          ?        0
Impacted Files Coverage Δ
src/kinesis/producer.py 32.18% <26.47%> (ø)

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 9cc9dec...e960634. Read the comment docs.

@borgstrom borgstrom closed this Dec 30, 2017
@borgstrom borgstrom reopened this Dec 30, 2017
@borgstrom borgstrom changed the title Project improvements Packaging improvements / Improve Producer interface Dec 31, 2017
@borgstrom borgstrom changed the title Packaging improvements / Improve Producer interface Packaging improvements / More producer control Dec 31, 2017
@borgstrom borgstrom merged commit 2ed1c1d into master Dec 31, 2017
@borgstrom borgstrom deleted the project-improvements branch December 31, 2017 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants