You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/06/23 04:16:37 UTC

[GitHub] [pulsar] hangc0276 opened a new pull request #11030: fix pulsar-broker individual module build failed

hangc0276 opened a new pull request #11030:
URL: https://github.com/apache/pulsar/pull/11030


   ### Motivation
   When build pulsar-broker individual module as the folllowing step.
   ```bash
   cd pulsar-broker
   mvn clean install -DskipTests
   ```
   It will get the following exceptions.
   ```
   [ERROR] Failed to generate lightproto code for [/home/hangc/Downloads/tmp/pulsar/pulsar-broker/src/main/proto/TransactionPendingAck.proto, /home/hangc/Downloads/tmp/pulsar/pulsar-broker/src/main/proto/ResourceUsage.proto]: java.lang.IllegalStateException: Imported proto pulsar-common/src/main/proto/PulsarApi.proto not found. (null)
   java.lang.RuntimeException: java.lang.IllegalStateException: Imported proto pulsar-common/src/main/proto/PulsarApi.proto not found. (null)
       at io.protostuff.parser.Proto.importProto (Proto.java:316)
       at io.protostuff.parser.ProtoParser.header_import (ProtoParser.java:1431)
       at io.protostuff.parser.ProtoParser.statement (ProtoParser.java:336)
       at io.protostuff.parser.ProtoParser.parse (ProtoParser.java:165)
       at io.protostuff.parser.ProtoUtil.loadFrom (ProtoUtil.java:52)
       at io.protostuff.parser.ProtoUtil.loadFrom (ProtoUtil.java:60)
       at io.protostuff.parser.ProtoUtil.loadFrom (ProtoUtil.java:90)
       at com.github.splunk.lightproto.generator.LightProtoGenerator.generate (LightProtoGenerator.java:42)
       at com.github.splunk.lightproto.maven.plugin.LightProtoMojo.generate (LightProtoMojo.java:62)
       at com.github.splunk.lightproto.maven.plugin.LightProtoMojo.execute (LightProtoMojo.java:94)
       at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
       at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:210)
       at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
       at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
       at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
       at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
       at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
       at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
       at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
       at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
       at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
   ```
   The reason is `TransactionPendingAck.proto` uses project base dir to import `PulsarApi.proto`, which will lead to `PulsarApi.proto` not found when build in pulsar-broker module.
   
   ### Modification
   1. use relative path to import `PulsarApi.proto`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] BewareMyPower commented on pull request #11030: fix pulsar-broker individual module build failed

Posted by GitBox <gi...@apache.org>.
BewareMyPower commented on pull request #11030:
URL: https://github.com/apache/pulsar/pull/11030#issuecomment-866556253


   Could you change your description since it only modified README now.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] sijie merged pull request #11030: Update the instructions for building individual modules in README

Posted by GitBox <gi...@apache.org>.
sijie merged pull request #11030:
URL: https://github.com/apache/pulsar/pull/11030


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] hangc0276 commented on pull request #11030: fix pulsar-broker individual module build failed

Posted by GitBox <gi...@apache.org>.
hangc0276 commented on pull request #11030:
URL: https://github.com/apache/pulsar/pull/11030#issuecomment-866523550


   After use relative path for proto import, it will cause the project build failed when build in root path. So i update the `README.md` doc guide for individual module build and run individual test.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org