Skip to content

Pull upstream PyQS changes into our repo #1

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.

Sign up for GitHub

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

Jump to bottom
Open
aiera-devops wants to merge 58 commits into
base: master
Choose a base branch
from
Open

Pull upstream PyQS changes into our repo #1

aiera-devops wants to merge 58 commits into from

Conversation

@aiera-devops
Copy link

@aiera-devops aiera-devops commented Mar 21, 2024

Trello card link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Security

Acceptance

https://www.notion.so/aiera/Test-Flows-Template-5203736ccfcb424c81643065f2bbf5e2

  • I have reviewed the Test Flows template and it is up-to-date after considering my changes

jimjshields and others added 30 commits June 9, 2020 17:56
@jimjshields
Add hook getter and setter to the class
7c2b9e0
@jimjshields
Add hook context and call hooks
0ce9364
@jimjshields
Change API to be a global event registry that can handle multiple hoo…
750fb45 
…ks per event
@jimjshields
Clean up events API & add unit tests
f557af1
@jimjshields
Refactor running hooks & add functional tests
d2d19a7
@jimjshields
Fix style & Python 2 issues
0d456c0
@spulec
Merge pull request #71 from jimjshields/feature/add-pre-and-post-proc…
b0415a2 
…ess-hooks

Feature/add pre and post process hooks
@spulec
0.1.5
5df7bbc
@spulec
Fix rst.
d6c88f3
@spulec
Refactor connection creation.
b1eeed9
@spulec
Merge pull request #75 from spulec/cleanup-boto-conns
defb07f 
Refactor boto connection creation
@spulec
0.1.6
5be3712
@spulec
Cleanup make publish.
d0c8e3d
@psarma89
Add initial SimpleProcessWorker implementation
4889961
@psarma89
Add SimpleManagerWorker intial implementation
9bf631a
@psarma89
Add a simple-worker flag to use the SimpleManagerWorker class
c9bb542
@psarma89
Fix manager worker tests
f3af494
@psarma89
Remove visibility timeout check from SimpleProcessWorkerbut keep the …
394246d 
…visibility timeout info
@psarma89
Default batchsize to be 1 if simple-worker flag is passed
c2041a1
@psarma89
Rename SimpleManagerWorker to BaseManager
e02acd2
@psarma89
Subclass SimpleManagerWorker and ManagerWorker from BaseManager
1ef4c9f
@psarma89
Update BaseManager to SimpleManagerWorker for main
ab0d385
@psarma89
Move SimpleProcessWorker methods to SimpleManagerWorker sub class
fdd9d40
@psarma89
Add methods to BaseManager that sub classes should override and imple…
f0e98b3 
…ment
@psarma89
Move worker_children state down to the subclasses
f3b9d66
@psarma89
Make replace_workers something that subclasses have to implement
e587c13
@psarma89
Move up _run_hooks to the parent class BaseWorker
0c49f79
@psarma89
Move common pre_process_context code from child classes to BaseWorker…
4bafcfa 
… parent
@psarma89
Move messages_processed into instance state for ProcessWorker
fbc716e
@psarma89
Add interval back to Simple* classes
891150c
psarma89 and others added 28 commits April 15, 2021 11:06
@psarma89
Refactor ProcessWorker and SimpleProcessWorker to better align proces…
4194526 
…s_message
@psarma89
Move _process_task into parent BaseWorker
ad710a9
@psarma89
Created a new base class BaseProcessWorker so that we don't give Read…
0a06269 
…Worker methods it doesn't need
@psarma89
Fix test_master_shuts_down_busy_process_workers to look at worker pro…
c5411e3 
…cesses
@psarma89
Add tests for SimpleManagerWorker
e81e2de
@psarma89
Add tests for different default batchsize for simple worker vs normal…
407dcbe 
… worker
@psarma89
Fix pre-commit issue
f62bafe
@psarma89
Revert test test_master_shuts_down_busy_process_workers to original
b8d63ee
@psarma89
Don't run python 2 travis workflow
bcfdcc1
@spulec
Merge pull request #76 from psarma89/feature/simple-process-worker
44da356 
Feature/simple process worker
@spulec
1.0.0
01f43cd
@psarma89
Update README for simple process worker
e8a564e
@spulec
Merge pull request #77 from psarma89/simple-process-worker-update-readme
cf5f25a 
Initial README update for simple process worker
@januszm
Make worker_children class configurable in manager subclasses
229ebd2
@januszm
Add Github Action for linter and tests
b8d5ec1
@januszm
Properly ignore flake8 with noqa
d0f0dac
@januszm
Ignore VIRTUAL_ENV check in CI
958f9d6
@spulec
Merge pull request #78 from januszm/fix/configurable_worker_classes
0488e07 
Make worker_children class configurable in manager subclasses
@psarma89
Add message_id to pre_process_context
85a711f
@spulec
Merge pull request #80 from psarma89/feature/add-message-id-to-context
1b22c9d 
Add message_id to pre_process_context
@spulec
1.0.1
1aeef3b
@opapy
add --endpint-url params
05c7487 
add --endpint-url params
@spulec
Merge pull request #81 from opapy/feature/add-endpoint-url-params
a983ce1 
add --endpint-url params
@filipemcg
Fix typos in README file
3f2b2f0
@spulec
Merge pull request #82 from filipemcg/master
7fce9af 
Fix typos in README file
@psarma89
As a starting point fix tests that were broken by PR #81 for endpoint…
8852017 
…_url
@psarma89
Removed introduced formatting
272fd18
@spulec
Merge pull request #85 from psarma89/bug/fix-tests-broken-by-endpoint…
b26e9f5 
…-url

As a starting point fix tests that were broken by PR #81 for endpoint_url
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@aiera-devops @jimjshields @spulec @psarma89 @januszm @opapy @filipemcg