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

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

andersonm-ibm opened a new pull request #878:
URL: https://github.com/apache/parquet-mr/pull/878


   This PR is another option of solving the problem "Cannot build from tarball 
because of git submodules".
   1. Encryption interop tests run separately from unit tests - in 
integration-test phase 
   2. Files for interop tests are downloaded manually, so git submodules are 
totally removed from the project
   
   - `mvn package/install` - doesn't run interop tests
   - `mvn verify` - run interop test and download files for interop from GitHub 
directly


----------------------------------------------------------------
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:
us...@infra.apache.org


> Cannot build from tarball because of git submodules
> ---------------------------------------------------
>
>                 Key: PARQUET-1992
>                 URL: https://issues.apache.org/jira/browse/PARQUET-1992
>             Project: Parquet
>          Issue Type: Bug
>            Reporter: Gabor Szadovszky
>            Priority: Blocker
>
> Because we use git submodules (to get test parquet files) a simple "mvn clean 
> install" fails from the unpacked tarball due to "not a git repository".
> I think we would have 2 options to solve this situation:
> * Include all the required files (even only for testing) in the tarball and 
> somehow avoid the git submodule update in case of executed in a non-git 
> envrionment
> * Make the downloading of the parquet files and the related tests optional so 
> it won't fail the build from the tarball



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

Reply via email to