Bigtop
  1. Bigtop
  2. BIGTOP-164

lzo codecs need to be disabled everywhere in Bigtop

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.1.0
    • Fix Version/s: 0.2.0
    • Component/s: General
    • Labels:
      None
    1. BIGTOP-164.patch.txt
      1.0 kB
      Roman Shaposhnik
    2. BIGTOP-164-1.patch.txt
      1 kB
      Roman Shaposhnik

      Activity

      Roman Shaposhnik created issue -
      Roman Shaposhnik made changes -
      Field Original Value New Value
      Attachment BIGTOP-164.patch.txt [ 12500913 ]
      Roman Shaposhnik made changes -
      Status Open [ 1 ] Patch Available [ 10002 ]
      Hide
      Bruno Mahé added a comment -

      So there is no value in keeping the gzip around?

      Show
      Bruno Mahé added a comment - So there is no value in keeping the gzip around?
      Hide
      Peter Linnell added a comment -

      Would this be better commented out, so it can optionally be enabled ?

      Show
      Peter Linnell added a comment - Would this be better commented out, so it can optionally be enabled ?
      Hide
      Roman Shaposhnik added a comment -

      The default values for io.compression.codes already cover the ones that are freely available:
      http://hadoop.apache.org/common/docs/current/core-default.html

      Show
      Roman Shaposhnik added a comment - The default values for io.compression.codes already cover the ones that are freely available: http://hadoop.apache.org/common/docs/current/core-default.html
      Hide
      Roman Shaposhnik added a comment -

      As for commenting out – I think I like this idea. We are probably going to enable snappy in Bigtop at some point so that'll be pretty useful.

      Show
      Roman Shaposhnik added a comment - As for commenting out – I think I like this idea. We are probably going to enable snappy in Bigtop at some point so that'll be pretty useful.
      Roman Shaposhnik made changes -
      Attachment BIGTOP-164-1.patch.txt [ 12500932 ]
      Hide
      Bruno Mahé added a comment -

      +1

      Show
      Bruno Mahé added a comment - +1
      Hide
      Peter Linnell added a comment -

      + 1 LGTM

      Show
      Peter Linnell added a comment - + 1 LGTM
      Roman Shaposhnik made changes -
      Status Patch Available [ 10002 ] Resolved [ 5 ]
      Resolution Fixed [ 1 ]
      Gavin made changes -
      Workflow no-reopen-closed, patch-avail [ 12639493 ] patch-available, re-open possible [ 12665840 ]
      Roman Shaposhnik made changes -
      Status Resolved [ 5 ] Closed [ 6 ]

        People

        • Assignee:
          Roman Shaposhnik
          Reporter:
          Roman Shaposhnik
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development