Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-802

No partition columns in WEB catalog page.

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.0, 0.8.1
    • Component/s: None
    • Labels:
      None

      Description

      See title.
      There is no partition columns in catalogview.jsp.

      1. TAJO-802.patch
        2 kB
        Hyoungjun Kim
      2. tajo_web_partition.png
        125 kB
        Hyoungjun Kim

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #205 (See https://builds.apache.org/job/Tajo-master-build/205/)
        TAJO-802: No partition columns in WEB catalog page. (Hyoungjun Kim via hyunsik) (hyunsik: rev 6cc5006d5ffbc88aa7b34c18d762c69f8c347418)

        • CHANGES
        • tajo-core/src/main/resources/webapps/admin/catalogview.jsp
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #205 (See https://builds.apache.org/job/Tajo-master-build/205/ ) TAJO-802 : No partition columns in WEB catalog page. (Hyoungjun Kim via hyunsik) (hyunsik: rev 6cc5006d5ffbc88aa7b34c18d762c69f8c347418) CHANGES tajo-core/src/main/resources/webapps/admin/catalogview.jsp
        Hide
        hyunsik Hyunsik Choi added a comment -

        committed it to master and 0.8.1.

        Show
        hyunsik Hyunsik Choi added a comment - committed it to master and 0.8.1.
        Hide
        hyunsik Hyunsik Choi added a comment -

        +1

        The patch looks good to me. This patch only changes catalog.jsp file. So, it is hard to contain unit tests. We don't need to consider TAJO QA's -1. But, I don't know why patch application was failed. In these days, jenkins is too slow and it is hard to access the jenkins WEB UI. So, I'll take a look at the jenkins problem.

        I'll commit it shortly.

        Show
        hyunsik Hyunsik Choi added a comment - +1 The patch looks good to me. This patch only changes catalog.jsp file. So, it is hard to contain unit tests. We don't need to consider TAJO QA's -1. But, I don't know why patch application was failed. In these days, jenkins is too slow and it is hard to access the jenkins WEB UI. So, I'll take a look at the jenkins problem. I'll commit it shortly.
        Hide
        tajoqa Tajo QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12642297/tajo_web_partition.png
        against master revision 8321d26.

        -1 patch. The patch command could not apply the patch.

        +1 @author. The patch does not contain any @author tags.

        -1 tests included. The patch doesn't appear to include any new or modified tests.
        Please justify why no new tests are needed for this patch.
        Also please list what manual steps were performed to verify this patch.

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/392//console

        This message is automatically generated.

        Show
        tajoqa Tajo QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12642297/tajo_web_partition.png against master revision 8321d26. -1 patch. The patch command could not apply the patch. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/392//console This message is automatically generated.

          People

          • Assignee:
            hjkim Hyoungjun Kim
            Reporter:
            hjkim Hyoungjun Kim
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development