Skip to content

Commit

Permalink
Add build param to skip creating hardlinks
Browse files Browse the repository at this point in the history
Presto build fails when a filesystem with no support for hardlinks is used. Add new param to specify hardlink include jars.
  • Loading branch information
singcha committed Oct 7, 2024
1 parent 6f60355 commit 8e6e5b5
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 1 deletion.
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,10 @@ Presto has a comprehensive set of unit tests that can take several minutes to ru

./mvnw clean install -DskipTests

Presto uses provisio plugin for packaging using hard links. If your filesystem doesn't support hardlinks, you can bypass hardlinks using:

./mvnw clean install -presto.hard-link-includes=''

After building Presto for the first time, you can load the project into your IDE and run the server. We recommend using [IntelliJ IDEA](http://www.jetbrains.com/idea/). Because Presto is a standard Maven project, you can import it into your IDE using the root `pom.xml` file. In IntelliJ, choose Open Project from the Quick Start box or choose Open from the File menu and select the root `pom.xml` file.

After opening the project in IntelliJ, double check that the Java SDK is properly configured for the project:
Expand Down
1 change: 1 addition & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@
<grpc.version>1.64.0</grpc.version>

<air.javadoc.lint>-missing</air.javadoc.lint>
<presto.hard-link-includes>**/*.jar</presto.hard-link-includes>
</properties>

<modules>
Expand Down
1 change: 1 addition & 0 deletions presto-server/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
<process-name>${project.artifactId}</process-name>

<skipTakariLifecyclePlugin>false</skipTakariLifecyclePlugin>
<presto.hard-link-includes>${project.parent.presto.hard-link-includes}</presto.hard-link-includes>
</properties>

<build>
Expand Down
2 changes: 1 addition & 1 deletion presto-server/src/main/provisio/presto.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<runtime>
<!-- Target -->
<archive name="${project.artifactId}-${project.version}.tar.gz" hardLinkIncludes="**/*.jar" />
<archive name="${project.artifactId}-${project.version}.tar.gz" hardLinkIncludes="${presto.hard-link-includes}" />

<!-- Notices -->
<fileSet to="/">
Expand Down

0 comments on commit 8e6e5b5

Please sign in to comment.