[GitHub] [hadoop] jojochuang commented on a change in pull request #763: [WIP] HADOOP-15984. Update jersey from 1.19 to 2.x
jojochuang commented on a change in pull request #763: [WIP] HADOOP-15984. Update jersey from 1.19 to 2.x URL: https://github.com/apache/hadoop/pull/763#discussion_r312630346 ## File path: hadoop-common-project/hadoop-kms/src/main/conf/kms-log4j.properties ## @@ -34,7 +34,6 @@ log4j.additivity.kms-audit=false log4j.rootLogger=INFO, kms log4j.logger.org.apache.hadoop=INFO -log4j.logger.com.sun.jersey.server.wadl.generators.WadlGeneratorJAXBGrammarGenerator=OFF Review comment: This is related to HADOOP-15686. If I get HADOOP-15686 in, we'll have to update here. 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 With regards, Apache Git Services - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[GitHub] [hadoop] jojochuang commented on a change in pull request #763: [WIP] HADOOP-15984. Update jersey from 1.19 to 2.x
jojochuang commented on a change in pull request #763: [WIP] HADOOP-15984. Update jersey from 1.19 to 2.x URL: https://github.com/apache/hadoop/pull/763#discussion_r312631751 ## File path: hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/web/resources/UserProvider.java ## @@ -1,73 +1,37 @@ -/** Review comment: why was the license text removed? 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 With regards, Apache Git Services - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[GitHub] [hadoop] jojochuang commented on a change in pull request #763: [WIP] HADOOP-15984. Update jersey from 1.19 to 2.x
jojochuang commented on a change in pull request #763: [WIP] HADOOP-15984.
Update jersey from 1.19 to 2.x
URL: https://github.com/apache/hadoop/pull/763#discussion_r312630777
##
File path: hadoop-common-project/hadoop-kms/src/test/resources/log4j.properties
##
@@ -25,7 +25,6 @@ log4j.appender.stdout.layout.ConversionPattern=%d{ISO8601}
%-5p %c{1} - %m%n
log4j.rootLogger=INFO, stdout
log4j.logger.org.apache.hadoop.conf=ERROR
log4j.logger.org.apache.hadoop.crytpo.key.kms.server=ALL
-log4j.logger.com.sun.jersey.server.wadl.generators.WadlGeneratorJAXBGrammarGenerator=OFF
Review comment:
TODO: HADOOP-15686
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
With regards,
Apache Git Services
-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[GitHub] [hadoop] jojochuang commented on a change in pull request #763: [WIP] HADOOP-15984. Update jersey from 1.19 to 2.x
jojochuang commented on a change in pull request #763: [WIP] HADOOP-15984. Update jersey from 1.19 to 2.x URL: https://github.com/apache/hadoop/pull/763#discussion_r312216297 ## File path: hadoop-project/pom.xml ## @@ -66,6 +66,7 @@ 1.19 +2.28 Review comment: Can we bump it to 2.29 to support JDK 11? http://blog.supol.cz/?p=144 Jersey 2.29 supports JDK11 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 With regards, Apache Git Services - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
