Skip to content

Conversation

@Fokko
Copy link
Contributor

@Fokko Fokko commented Jan 21, 2019

Copy link
Contributor

@gszadovszky gszadovszky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for this fix. Please, check my comments.

@Fokko Fokko force-pushed the fd-add-java-11-builds branch from e401cfa to d4e8ebb Compare January 22, 2019 10:11
Copy link
Contributor

@gszadovszky gszadovszky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Waiting for the fixes of the java11 related issues. I'll push it if Travis tests pass.

@Fokko
Copy link
Contributor Author

Fokko commented Jan 30, 2019

Rebased.

@Fokko Fokko force-pushed the fd-add-java-11-builds branch from a103088 to 7ebbfb7 Compare February 5, 2019 22:05
@Fokko
Copy link
Contributor Author

Fokko commented Feb 10, 2019

Thrift was using Gradle 4.4.1, but has been updated to 5.1.1 which supports Java11:
apache/thrift@98f379e#diff-7762a898c27a3d29c23245cc2abd99baR5

@Fokko Fokko force-pushed the fd-add-java-11-builds branch from 7ebbfb7 to 54d8490 Compare May 10, 2019 11:14
@zivanfi zivanfi changed the title [PARQUET-1499] Enable JDK11 build PARQUET-1499: Enable JDK11 build Jun 11, 2019
@zivanfi zivanfi changed the title PARQUET-1499: Enable JDK11 build PARQUET-1499: Add Java 11 to Travis Jun 11, 2019
@Fokko Fokko force-pushed the fd-add-java-11-builds branch 4 times, most recently from 71c35c9 to e9485ab Compare June 19, 2019 09:48
@Fokko Fokko force-pushed the fd-add-java-11-builds branch from e9485ab to 9da1fe1 Compare June 19, 2019 11:18
@Fokko
Copy link
Contributor Author

Fokko commented Jun 19, 2019

I've split some of the stuff in this PR into #656 and #654 to make the PR's nice and manageable. For this PR, we're still dependent on #605. After updating Scala, I expect the CI to pass.

Fokko Driesprong and others added 11 commits July 4, 2019 11:12
Got some weird warnings from generated code:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-javadoc-plugin:2.9.1:javadoc (default-cli) on project parquet-format-structures: An error has occurred in JavaDocs report generation:
[ERROR] Exit code: 1 - javadoc: error - The code being documented uses modules but the packages defined in http://docs.oracle.com/javase/7/docs/api/ are in the unnamed module.
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:49: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:394: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:407: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:84: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:21: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:150: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:159: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:29: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:49: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:21: warning - invalid usage of tag <
So we can run it in Docker as well
@Fokko Fokko force-pushed the fd-add-java-11-builds branch from 9da1fe1 to ad4697e Compare July 4, 2019 09:14
@Fokko
Copy link
Contributor Author

Fokko commented Oct 25, 2019

Rebased onto master and it is green now :-)

@Fokko Fokko merged commit 19b10ac into apache:master Oct 25, 2019
@Fokko Fokko deleted the fd-add-java-11-builds branch October 25, 2019 23:06
shangxinli pushed a commit to shangxinli/parquet-mr that referenced this pull request Dec 16, 2019
Got some weird warnings from generated code:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-javadoc-plugin:2.9.1:javadoc (default-cli) on project parquet-format-structures: An error has occurred in JavaDocs report generation:
[ERROR] Exit code: 1 - javadoc: error - The code being documented uses modules but the packages defined in http://docs.oracle.com/javase/7/docs/api/ are in the unnamed module.
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:49: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:394: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:407: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:84: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:21: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:150: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:159: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:29: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/ColumnIndex.java:49: warning - invalid usage of tag <
[ERROR] /home/travis/build/apache/parquet-mr/parquet-format-structures/target/generated-sources/thrift/org/apache/parquet/format/OffsetIndex.java:21: warning - invalid usage of tag <
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants