Skip to content

Commit

Permalink
[FLINK-2544] [docs] Add Java 8 version for building PowerMock tests t…
Browse files Browse the repository at this point in the history
…o docs

Java 8 update 11 introduced a stricter bytecode verifier that leads
to failures in unit tests that use the PowerMock runner. The tests
run correctly in Java 8u51 or above.

This closes #1882
  • Loading branch information
skavulya authored and fhueske committed Apr 15, 2016
1 parent dc78a74 commit d938c5f
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,8 @@ mvn clean package -DskipTests # this will take up to 10 minutes

Flink is now installed in `build-target`

*NOTE: Maven 3.3.x can build Flink, but will not properly shade away certain dependencies. Maven 3.0.3 creates the libraries properly.*
*NOTE: Maven 3.3.x can build Flink, but will not properly shade away certain dependencies. Maven 3.0.3 creates the libraries properly.
To build unit tests with Java 8, use Java 8u51 or above to prevent failures in unit tests that use the PowerMock runner.*

## Developing Flink

Expand Down
3 changes: 3 additions & 0 deletions docs/setup/building.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,9 @@ In order to build Flink you need the source code. Either [download the source of

In addition you need **Maven 3** and a **JDK** (Java Development Kit). Flink requires **at least Java 7** to build. We recommend using Java 8.

*NOTE: Maven 3.3.x can build Flink, but will not properly shade away certain dependencies. Maven 3.0.3 creates the libraries properly.
To build unit tests with Java 8, use Java 8u51 or above to prevent failures in unit tests that use the PowerMock runner.*

To clone from git, enter:

~~~bash
Expand Down

0 comments on commit d938c5f

Please sign in to comment.