Skip to content

Conversation

ikavgo
Copy link
Contributor

@ikavgo ikavgo commented May 22, 2023

Proposed Changes

  1. remove per consumer timeout
  2. add shortcut for queue policy with help string
  3. add column to consumer table on queue page

Types of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply

  • Bug fix (non-breaking change which fixes issue #NNNN)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause an observable behavior change in existing systems)
  • Documentation improvements (corrections, new content, etc)
  • Cosmetic change (whitespace, formatting, etc)
  • Build system and/or CI

Checklist

Put an x in the boxes that apply.
You can also fill these out after creating the PR.
If you're unsure about any of them, don't hesitate to ask on the mailing list.
We're here to help!
This is simply a reminder of what we are going to look for before merging your code.

Further Comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc.

@ikavgo ikavgo changed the title Ik consumer timeout followup Consumer Timeout Follow-up May 22, 2023
@michaelklishin michaelklishin added this to the 3.11.17 milestone May 22, 2023
@ikavgo ikavgo force-pushed the ik-consumer-timeout-followup branch from 8ebb6fb to 00b3a89 Compare May 22, 2023 09:59
@ikavgo ikavgo requested a review from michaelklishin May 22, 2023 11:37
@michaelklishin michaelklishin modified the milestones: 3.11.17, 3.12.0 May 22, 2023
@michaelklishin michaelklishin merged commit e569a2b into main May 22, 2023
@michaelklishin michaelklishin deleted the ik-consumer-timeout-followup branch May 22, 2023 20:28
michaelklishin added a commit that referenced this pull request May 22, 2023
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.

3 participants