You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tez.apache.org by GitBox <gi...@apache.org> on 2021/02/12 03:26:30 UTC
[GitHub] [tez] belugabehr opened a new pull request #110: TEZ-4289: Remove Dependency on commons-math
belugabehr opened a new pull request #110:
URL: https://github.com/apache/tez/pull/110
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] belugabehr commented on pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
belugabehr commented on pull request #110:
URL: https://github.com/apache/tez/pull/110#issuecomment-780833760
@jteagles @abstractdog last test run passed, but results were only published to JIRA and not to Github. Please consider for merge.
https://issues.apache.org/jira/browse/TEZ-4289?focusedCommentId=17286075&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17286075
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] belugabehr closed pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
belugabehr closed pull request #110:
URL: https://github.com/apache/tez/pull/110
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] belugabehr closed pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
belugabehr closed pull request #110:
URL: https://github.com/apache/tez/pull/110
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] belugabehr commented on pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
belugabehr commented on pull request #110:
URL: https://github.com/apache/tez/pull/110#issuecomment-780832874
@abstractdog Not sure what's up this time. Seem to be getting a 404 error on the `continuous-integration/jenkins/pr-merge`. Any thoughts?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] jteagles merged pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
jteagles merged pull request #110:
URL: https://github.com/apache/tez/pull/110
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] jteagles commented on pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
jteagles commented on pull request #110:
URL: https://github.com/apache/tez/pull/110#issuecomment-780874699
+1. Think this is good.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] jteagles commented on pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
jteagles commented on pull request #110:
URL: https://github.com/apache/tez/pull/110#issuecomment-778541193
I think jdk 11 build was enabled today. @abstractdog can you verify? If so, I think we will need fix from
https://issues.apache.org/jira/browse/TEZ-4025
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] abstractdog commented on pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
abstractdog commented on pull request #110:
URL: https://github.com/apache/tez/pull/110#issuecomment-778548249
> I think jdk 11 build was enabled today. @abstractdog can you verify? If so, I think we will need fix from
> https://issues.apache.org/jira/browse/TEZ-4025
yes, I think javadoc jdk11 issues should be ignored now and in the near future, while we figure out and a merge a fix for that
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] abstractdog edited a comment on pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
abstractdog edited a comment on pull request #110:
URL: https://github.com/apache/tez/pull/110#issuecomment-778548249
> I think jdk 11 build was enabled today. @abstractdog can you verify? If so, I think we will need fix from
> https://issues.apache.org/jira/browse/TEZ-4025
yes, I think javadoc jdk11 issues should be ignored now and in the near future, while we figure out and merge a fix for that
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [tez] belugabehr commented on pull request #110: TEZ-4289: Remove Dependency on commons-math
Posted by GitBox <gi...@apache.org>.
belugabehr commented on pull request #110:
URL: https://github.com/apache/tez/pull/110#issuecomment-778539967
Everything looks good here, but getting a weird build error. Not sure what to make of it:
```none
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-javadoc-plugin:3.0.1:javadoc (default-cli) on project tez-javadoc-tools: An error has occurred in Javadoc report generation:
[ERROR] Exit code: 1 - /home/jenkins/jenkins-agent/workspace/tez-multibranch_PR-110/src/tez-tools/tez-javadoc-tools/src/main/java/org/apache/tez/tools/javadoc/doclet/ConfigStandardDoclet.java:36: warning: [removal] AnnotationDesc in com.sun.javadoc has been deprecated and marked for removal
[ERROR] import com.sun.javadoc.AnnotationDesc.ElementValuePair;
[ERROR] ^
[ERROR] /home/jenkins/jenkins-agent/workspace/tez-multibranch_PR-110/src/tez-tools/tez-javadoc-tools/src/main/java/org/apache/tez/tools/javadoc/doclet/ConfigStandardDoclet.java:42: error: package com.sun.tools.doclets.standard is not visible
[ERROR] import com.sun.tools.doclets.standard.Standard;
[ERROR] ^
[ERROR] (package com.sun.tools.doclets.standard is declared in module jdk.javadoc, which does not export it)
```
----------------------------------------------------------------
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:
users@infra.apache.org