Hive
  1. Hive
  2. HIVE-5205

Javadoc warnings in HCatalog prevent Hive from building under OpenJDK7

    Details

      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

          Hide
          Konstantin Boudnik added a comment -

          That's do the trick.

          Show
          Konstantin Boudnik added a comment - That's do the trick.
          Hide
          Brock Noland added a comment -

          +1

          Show
          Brock Noland added a comment - +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.
          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
          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 -

          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?
          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 -

          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 -

          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 .

            People

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

              Dates

              • Created:
                Updated:

                Development