[ 
https://issues.apache.org/jira/browse/PARQUET-1863?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17110273#comment-17110273
 ] 

ASF GitHub Bot commented on PARQUET-1863:
-----------------------------------------

gszadovszky merged pull request #792:
URL: https://github.com/apache/parquet-mr/pull/792


   


----------------------------------------------------------------
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:
[email protected]


> Remove use of add-test-source mojo in parquet-protobuf
> ------------------------------------------------------
>
>                 Key: PARQUET-1863
>                 URL: https://issues.apache.org/jira/browse/PARQUET-1863
>             Project: Parquet
>          Issue Type: Improvement
>          Components: parquet-mr
>            Reporter: Laurent Goujon
>            Priority: Minor
>
> {{parquet-protobuf}} uses {{build-helper-maven-plugin:add-test-source}} maven 
> mojo to add protobuf test classes to the test sources path, but 
> {{protoc-jar-maven-plugin}} also adds these classes to the main sources path. 
> It is unnecessary ({{protoc-jar-maven-plugin}} could be configured to add to 
> the test classes path) and actually causes confusion with some IDE (Eclipse).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to