trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From traflm <...@git.apache.org>
Subject [GitHub] trafodion pull request #1549: [TRAFODION-3054] update pom.xml.cdh for hbase-...
Date Sat, 05 May 2018 21:36:37 GMT
GitHub user traflm opened a pull request:

    https://github.com/apache/trafodion/pull/1549

    [TRAFODION-3054] update pom.xml.cdh for hbase-trx to build protobuf

    The various pom.xml.cdh<version> file can be used as pom.xml for maven input, to
build protobuf java output for hbase-trx.
    As we want to modify the .proto to add a new coprocessor, we found that it failed with
ERROR:
    
    Caused by: java.lang.UnsupportedClassVersionError: org/apache/hadoop/maven/plugin/protoc/ProtocMojo
: Unsupported major.minor version 52.0
    
    And the plug-in information is 
    
    [INFO] --- hadoop-maven-plugins:3.1.0:protoc (compile-protoc) @ hbase-trx-cdh5_4 ---
    
    So the lasted Hadoop maven plugins for protoc is 3.1.0 which need JDK 1.8.
    
    It also means the pom.xml.cdh<version> files should be updated to match the Hadoop-Maven-plugin
version same as its hadoop version, instead of using latest version (without specifying the
version tag)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/traflm/trafodion TRAFODION-3054

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafodion/pull/1549.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1549
    
----
commit 1cc728fa7eaf444e7dc6f61a4bd7ed65b12ad7e1
Author: Liu Ming <ovis_poly@...>
Date:   2018-05-05T16:15:40Z

    [TRAFODION-3054] update pom.xml.cdh for hbase-trx to build protobuf

----


---

Mime
View raw message