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

Added threading feature. #5

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

gautamkhanapuri
Copy link

When working on a project at my workplace, I found this library very useful.

I have added the threading feature. The user needs to create an instance of Subset of Consumer class. He can then call the .start_consumer() instance method which will intrinsically create a thread if the threaded variable is set to True . The user has the option to create a daemon or non-daemon thread.
The user need not worry about joining or stopping the thread manually at the time of program exit. It is done automatically. It can be done manually if they wish to.
The changes have passed all the tests. Let me know if I have to add any tests. I will change the documentation as soon as you say the changes are ok.
If there are any changes you would like me to make, please let me know.

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.

1 participant