Uploaded image for project: 'Samza'
  1. Samza
  2. SAMZA-956

Disk Quotas: Add throttler and disk quota enforcement

    Details

    • Type: New Feature
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.10.1
    • Component/s: None
    • Labels:
      None

      Description

      Continuation of Disk Quotas work started with SAMZA-924. In this patch we add disk quota enforcement using a set of policies that cause the run loop execution rate to be throttled.

      1. samza.patch
        88 kB
        Chris Pettitt
      2. samza-2.patch
        88 kB
        Chris Pettitt
      3. SAMZA-956.patch
        89 kB
        Navina Ramesh

        Activity

        Hide
        jmakes Jake Maes added a comment -
        Show
        jmakes Jake Maes added a comment - RB: https://reviews.apache.org/r/48080/
        Hide
        navina Navina Ramesh added a comment -

        Chris Pettitt Left some comments on the RB. Otherwise, +1 !

        Show
        navina Navina Ramesh added a comment - Chris Pettitt Left some comments on the RB. Otherwise, +1 !
        Hide
        cpettitt-linkedin Chris Pettitt added a comment -

        Navina, thanks for the comments. I've addressed both issues and attached a new patch (samza-2.patch). Should I update the RB too?

        Show
        cpettitt-linkedin Chris Pettitt added a comment - Navina, thanks for the comments. I've addressed both issues and attached a new patch (samza-2.patch). Should I update the RB too?
        Hide
        navina Navina Ramesh added a comment -

        I think it is redundant at this point. Let's skip it. I will try the new patch.

        Show
        navina Navina Ramesh added a comment - I think it is redundant at this point. Let's skip it. I will try the new patch.
        Hide
        cpettitt-linkedin Chris Pettitt added a comment -

        Updated the RB in the meantime if it is of any help.

        Show
        cpettitt-linkedin Chris Pettitt added a comment - Updated the RB in the meantime if it is of any help.
        Hide
        navina Navina Ramesh added a comment -

        Made some cosmetic changes to pass checkstyle

        Show
        navina Navina Ramesh added a comment - Made some cosmetic changes to pass checkstyle
        Hide
        navina Navina Ramesh added a comment -

        Committed the changes!

        Show
        navina Navina Ramesh added a comment - Committed the changes!

          People

          • Assignee:
            navina Navina Ramesh
            Reporter:
            cpettitt-linkedin Chris Pettitt
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development