Hive
  1. Hive
  2. HIVE-5205

Javadoc warnings in HCatalog prevent Hive from building under OpenJDK7

    Details

    • Release Note:
      Perhaps it is of no interest anymore, closing.

      Description

      when building Hive with OpenJDK7 the following warning message makes the build fail:

      [javadoc] /var/lib/jenkins/workspace/Shark-Hive-0.11-OJDK7/hcatalog/storage-handlers/hbase/src/java/org/apache/hcatalog/hbase/snapshot/RevisionManagerFactory.java:81: warning - @return tag has no arguments.

      1. HIVE-5205.patch
        0.9 kB
        Konstantin Boudnik

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          11h 27m 1 Brock Noland 04/Sep/13 12:18
          Patch Available Patch Available Open Open
          1h 12m 1 Brock Noland 04/Sep/13 13:31
          Open Open Resolved Resolved
          763d 6h 14m 1 Konstantin Boudnik 07/Oct/15 19:46
          Konstantin Boudnik made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Release Note Perhaps it is of no interest anymore, closing.
          Resolution Won't Fix [ 2 ]
          Hide
          Xuefu Zhang added a comment -

          In that case, I made the issue standalone and marked as "related to" HIVE-4583.

          Show
          Xuefu Zhang added a comment - In that case, I made the issue standalone and marked as "related to" HIVE-4583 .
          Xuefu Zhang made changes -
          Link This issue is related to HIVE-4583 [ HIVE-4583 ]
          Xuefu Zhang made changes -
          Parent HIVE-4583 [ 12648611 ]
          Issue Type Sub-task [ 7 ] Bug [ 1 ]
          Xuefu Zhang made changes -
          Rank Ranked higher
          Hide
          Konstantin Boudnik added a comment -

          I'd argue that grouping of issues is only loosely coupled with the versions of a product. Grouping is logical; where's versioning is functional. In this particular case, parent issue is about JDK7 support. This ticket is exactly about it. But I really don't care one way or another how they are going to be connected to each other, until they are linked in some way.

          Show
          Konstantin Boudnik added a comment - I'd argue that grouping of issues is only loosely coupled with the versions of a product. Grouping is logical; where's versioning is functional. In this particular case, parent issue is about JDK7 support. This ticket is exactly about it. But I really don't care one way or another how they are going to be connected to each other, until they are linked in some way.
          Hide
          Xuefu Zhang added a comment -

          I could be wrong, but my understanding is that, if there is patch that addresses the issue for the trunk, the patch may be backported to other branches for future releases.

          For this particular case, this issue was under HIVE-4583, which is for 0.12. So to say the least, the issue needs to be decoupled from the parent JIRA.

          Show
          Xuefu Zhang added a comment - I could be wrong, but my understanding is that, if there is patch that addresses the issue for the trunk, the patch may be backported to other branches for future releases. For this particular case, this issue was under HIVE-4583 , which is for 0.12. So to say the least, the issue needs to be decoupled from the parent JIRA.
          Hide
          Konstantin Boudnik added a comment -

          Shall we target it for 0.11.1 instead of simply closing it?

          Show
          Konstantin Boudnik added a comment - Shall we target it for 0.11.1 instead of simply closing it?
          Konstantin Boudnik made changes -
          Fix Version/s 0.11.1 [ 12324446 ]
          Fix Version/s 0.12.0 [ 12324312 ]
          Konstantin Boudnik made changes -
          Labels target-version_0.11
          Hide
          Xuefu Zhang added a comment -

          Konstantin Boudnik Brock Noland Should we close this?

          Show
          Xuefu Zhang added a comment - Konstantin Boudnik Brock Noland Should we close this?
          Hide
          Konstantin Boudnik added a comment -

          looks like it has been been fixed in the trunk as a part of
          0129f3642660e8696646ac3d4aa414ef062e2118
          Hence, the patch is only good for Hive 0.11

          Show
          Konstantin Boudnik added a comment - looks like it has been been fixed in the trunk as a part of 0129f3642660e8696646ac3d4aa414ef062e2118 Hence, the patch is only good for Hive 0.11
          Ashutosh Chauhan made changes -
          Assignee Konstantin Boudnik [ cos ]
          Brock Noland made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Hide
          Brock Noland added a comment -

          Doesn't look to apply any more.

          Show
          Brock Noland added a comment - Doesn't look to apply any more.
          Brock Noland made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Brock Noland added a comment -

          +1

          Show
          Brock Noland added a comment - +1
          Konstantin Boudnik made changes -
          Field Original Value New Value
          Attachment HIVE-5205.patch [ 12601294 ]
          Hide
          Konstantin Boudnik added a comment -

          That's do the trick.

          Show
          Konstantin Boudnik added a comment - That's do the trick.
          Konstantin Boudnik created issue -

            People

            • Assignee:
              Konstantin Boudnik
              Reporter:
              Konstantin Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development