Accumulo
  1. Accumulo
  2. ACCUMULO-1501

Trace table should have an age-off filter by default

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.5.0
    • Fix Version/s: 1.5.1, 1.6.0
    • Component/s: trace
    • Labels:
      None
    1. ACCUMULO-1501.patch
      4 kB
      Billie Rinaldi
    2. ACCUMULO-1501-2.patch
      2 kB
      Billie Rinaldi

      Activity

      Hide
      ASF subversion and git services added a comment -

      Commit 1494375 from Billie Rinaldi
      [ https://svn.apache.org/r1494375 ]

      ACCUMULO-1501 added age-off filter to trace table - merged to trunk

      Show
      ASF subversion and git services added a comment - Commit 1494375 from Billie Rinaldi [ https://svn.apache.org/r1494375 ] ACCUMULO-1501 added age-off filter to trace table - merged to trunk
      Hide
      ASF subversion and git services added a comment -

      Commit 1494372 from Billie Rinaldi
      [ https://svn.apache.org/r1494372 ]

      ACCUMULO-1501 added age-off filter to trace table

      Show
      ASF subversion and git services added a comment - Commit 1494372 from Billie Rinaldi [ https://svn.apache.org/r1494372 ] ACCUMULO-1501 added age-off filter to trace table
      Hide
      Keith Turner added a comment -

      I also prefer the simplicity of patch #2

      Show
      Keith Turner added a comment - I also prefer the simplicity of patch #2
      Hide
      Billie Rinaldi added a comment -

      Based on my concern about having multiple ways to set the property, I would favor the attached simpler patch. I also have one that uses a trace ttl property and re-applies it on restart, if others think that would be preferable.

      Show
      Billie Rinaldi added a comment - Based on my concern about having multiple ways to set the property, I would favor the attached simpler patch. I also have one that uses a trace ttl property and re-applies it on restart, if others think that would be preferable.
      Hide
      Billie Rinaldi added a comment -

      It seems like it could be confusing if someone configured the ttl manually on the trace table, then restarted the trace server and found that the ttl was reset/changed.

      Show
      Billie Rinaldi added a comment - It seems like it could be confusing if someone configured the ttl manually on the trace table, then restarted the trace server and found that the ttl was reset/changed.
      Hide
      Billie Rinaldi added a comment -

      Hmm. Do you think the trace ttl should be a configurable property, or would it be better to pick a default value and make users change the property on the table manually?

      Show
      Billie Rinaldi added a comment - Hmm. Do you think the trace ttl should be a configurable property, or would it be better to pick a default value and make users change the property on the table manually?
      Hide
      Eric Newton added a comment -

      The TTL iterator should be updated with the property value when it changes, or at start-up. But that implies that the trace user has that permission.

      Show
      Eric Newton added a comment - The TTL iterator should be updated with the property value when it changes, or at start-up. But that implies that the trace user has that permission.

        People

        • Assignee:
          Billie Rinaldi
          Reporter:
          Mike Drob
        • Votes:
          0 Vote for this issue
          Watchers:
          4 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development