-
Notifications
You must be signed in to change notification settings - Fork 206
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
Move some TQ triggers under new namespace. #1069
Conversation
…base-functions into dl-tasks-namespace
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Admittedly I scanned this one since much of it should be cut & paste.
@@ -0,0 +1,127 @@ | |||
// The MIT License (MIT) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: helper is a dirty word. Can we at least call it a fooHelper
? Maybe httpsBasedHelper
Move all task queue related API under newly formed
tasks
package. Task queue function are now undertasks
namespace:Same goes for handler namespace.
Few other changes:
maxRetrySeconds
option for task queues.maxBurstsize
option since it's outputonly.