-
Notifications
You must be signed in to change notification settings - Fork 234
Closed
Labels
api: pubsubIssues related to the googleapis/nodejs-pubsub API.Issues related to the googleapis/nodejs-pubsub API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Description
if changing flowControl settings after calling .on they dont not get into effect
const subscription = pubSubClient.subscription(subscriptionNameOrId);
const messageHandler = message => {
message.ack();
};
subscription.on('message', messageHandler);
// does not have any effect on flow control
subscription.setOptions({
flowControl: {
maxMessages: 1,
allowExcessMessages: true,
}
})
Metadata
Metadata
Assignees
Labels
api: pubsubIssues related to the googleapis/nodejs-pubsub API.Issues related to the googleapis/nodejs-pubsub API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.