Bigtop
  1. Bigtop
  2. BIGTOP-1357

Get rid of HCatalog packaging code

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 1.0.0
    • Component/s: debian, rpm
    • Labels:
      None

      Description

      HCatalog is bundled with Hive in the newer releases of hive. Consequently, it's packaging code should reside in Hive's packaging directories (and it already does).

      Consequently, we are not using https://github.com/apache/bigtop/tree/master/bigtop-packages/src/common/hcatalog
      and the corresponding locations in rpm and deb directories which used to package HCatalog before it was a part of hive.

      So, we should get rid of those directories. Also, we should check in bigtop.mk and make sure no references to building HCatalog remain. The only way, we should be able to build HCatalog should be via Hive.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        32d 8h 8m 1 Giridharan Kesavan 01/Aug/14 17:17
        In Progress In Progress Patch Available Patch Available
        64d 13h 7m 1 Giridharan Kesavan 05/Oct/14 06:24
        Patch Available Patch Available Resolved Resolved
        2d 16h 59m 1 Mark Grover 07/Oct/14 23:23
        Konstantin Boudnik made changes -
        Fix Version/s 1.0.0 [ 12326837 ]
        Fix Version/s 0.9.0 [ 12326836 ]
        Mark Grover made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Mark Grover added a comment -

        Committed. Thanks, Giri!

        Show
        Mark Grover added a comment - Committed. Thanks, Giri!
        Hide
        Peter Linnell added a comment -

        Thanks Mark for the clarification. +1 then as well.

        Show
        Peter Linnell added a comment - Thanks Mark for the clarification. +1 then as well.
        Hide
        Mark Grover added a comment -

        Peter, no change in hive packaging required since it already contains hcatalog packaging already: https://github.com/apache/bigtop/blob/master/bigtop-packages/src/rpm/hive/SPECS/hive.spec#L161

        The code being removed by this patch is just dead code.

        +1

        Show
        Mark Grover added a comment - Peter, no change in hive packaging required since it already contains hcatalog packaging already: https://github.com/apache/bigtop/blob/master/bigtop-packages/src/rpm/hive/SPECS/hive.spec#L161 The code being removed by this patch is just dead code. +1
        Hide
        Peter Linnell added a comment - - edited

        Giridharan Kesavan The patch looks good and I think it should definitely be part of 0.9.0. My question is do we need to change anything in the existing hive packaging ? Sorry for being a stickler, but would like to avoid unexpected breakage in 0.9.x

        Thanks!

        Show
        Peter Linnell added a comment - - edited Giridharan Kesavan The patch looks good and I think it should definitely be part of 0.9.0. My question is do we need to change anything in the existing hive packaging ? Sorry for being a stickler, but would like to avoid unexpected breakage in 0.9.x Thanks!
        Giridharan Kesavan made changes -
        Status In Progress [ 3 ] Patch Available [ 10002 ]
        Giridharan Kesavan made changes -
        Hide
        Giridharan Kesavan added a comment -

        this patch remove hcatalog pkg related changes

        Show
        Giridharan Kesavan added a comment - this patch remove hcatalog pkg related changes
        Hide
        Konstantin Boudnik added a comment -

        to 0.9.0 as no progress seems to be made so far.

        Show
        Konstantin Boudnik added a comment - to 0.9.0 as no progress seems to be made so far.
        Konstantin Boudnik made changes -
        Fix Version/s 0.9.0 [ 12326836 ]
        Fix Version/s 0.8.0 [ 12324841 ]
        Giridharan Kesavan made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Roman Shaposhnik made changes -
        Assignee Roman Shaposhnik [ rvs ] Giridharan Kesavan [ gkesavan ]
        Roman Shaposhnik made changes -
        Field Original Value New Value
        Assignee Roman Shaposhnik [ rvs ]
        Hide
        Giridharan Kesavan added a comment -

        Roman Shaposhnik
        Could you pls help assign this jira to me?

        Show
        Giridharan Kesavan added a comment - Roman Shaposhnik Could you pls help assign this jira to me?
        Hide
        Roman Shaposhnik added a comment -

        +1 to the idea.

        Show
        Roman Shaposhnik added a comment - +1 to the idea.
        Mark Grover created issue -

          People

          • Assignee:
            Giridharan Kesavan
            Reporter:
            Mark Grover
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development