[jira] [Comment Edited] (HADOOP-10075) Update jetty dependency to version 9

2016-10-28 Thread Robert Kanter (JIRA) 

[ 
https://issues.apache.org/jira/browse/HADOOP-10075?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15616515#comment-15616515
 ] 

Robert Kanter edited comment on HADOOP-10075 at 10/28/16 8:48 PM:
--

Just to clarify, the findbugs in the Hadoop QA report is misleading.  It found 
the previous findbugs error and gave a -1, but if you look lower, you'll see:
{quote}
hadoop-maven-plugins generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
{quote}
So it's actually fixed now, with the addendum 2 patch.


was (Author: rkanter):
Just to clarify, the findbugs in the Hadoop QA report is misleading.  It found 
the previous findbugs error and gave a -1, but if you look lower, you'll see:
{quote}
hadoop-maven-plugins generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
{quote}
So it's actually fixed now.

> Update jetty dependency to version 9
> 
>
> Key: HADOOP-10075
> URL: https://issues.apache.org/jira/browse/HADOOP-10075
> Project: Hadoop Common
>  Issue Type: Improvement
>Affects Versions: 2.2.0, 2.6.0
>Reporter: Robert Rati
>Assignee: Robert Kanter
>Priority: Critical
> Fix For: 3.0.0-alpha2
>
> Attachments: HADOOP-10075-002-wip.patch, HADOOP-10075.003.patch, 
> HADOOP-10075.004.patch, HADOOP-10075.005.patch, HADOOP-10075.006.patch, 
> HADOOP-10075.007.patch, HADOOP-10075.008.patch, HADOOP-10075.009.patch, 
> HADOOP-10075.010.patch, HADOOP-10075.011.patch, HADOOP-10075.patch, 
> HADOOP-10075_addendum.001.patch, HADOOP-10075_addendum.002.patch
>
>
> Jetty6 is no longer maintained.  Update the dependency to jetty9.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

[jira] [Comment Edited] (HADOOP-10075) Update jetty dependency to version 9

2016-10-27 Thread Brahma Reddy Battula (JIRA) 

[ 
https://issues.apache.org/jira/browse/HADOOP-10075?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15614079#comment-15614079
 ] 

Brahma Reddy Battula edited comment on HADOOP-10075 at 10/28/16 2:46 AM:
-

Compilation fails with following error, after this checked-in. Not going to 
revert, as the chnages are more..

 {noformat}
[ERROR] Failed to execute goal 
org.apache.hadoop:hadoop-maven-plugins:3.0.0-alpha2-SNAPSHOT:resource-gz 
(resource-gz) on
 project hadoop-common: org.apache.maven.plugin.MojoExecutionException: 
java.util.regex.PatternSyntaxException: Illegal/
unsupported escape sequence near index 3
[ERROR] 
D:\hadoop-trunk\hadoop\hadoop-common-project\hadoop-common\src\main\webapps\static
[ERROR] ^
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal 
org.apache.hadoop:hadoop-maven-plugins:3.
0.0-alpha2-SNAPSHOT:resource-gz (resource-gz) on project hadoop-common: 
org.apache.maven.plugin.MojoExecutionException:
java.util.regex.PatternSyntaxException: Illegal/unsupported escape sequence 
near index 3
D:\hadoop-trunk\hadoop\hadoop-common-project\hadoop-common\src\main\webapps\static
   ^
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:217)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:84)
at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:59)
at 
org.apache.maven.lifecycle.internal.LifecycleStarter.singleThreadedBuild(LifecycleStarter.java:183)
at 
org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:161)
at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:320)
at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:156)
at org.apache.maven.cli.MavenCli.execute(MavenCli.java:537)
at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:196)
at org.apache.maven.cli.MavenCli.main(MavenCli.java:141)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:290)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:230)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:409)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:352)
Caused by: org.apache.maven.plugin.MojoExecutionException: 
org.apache.maven.plugin.MojoExecutionException: java.util.reg
ex.PatternSyntaxException: Illegal/unsupported escape sequence near index 3
D:\OSCode\hadoop-trunk\hadoop\hadoop-common-project\hadoop-common\src\main\webapps\static
   ^
at 
org.apache.hadoop.maven.plugin.resourcegz.ResourceGzMojo.execute(ResourceGzMojo.java:82)
at 
org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:101)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:209)
... 19 more
Caused by: org.apache.maven.plugin.MojoExecutionException: 
java.util.regex.PatternSyntaxException: Illegal/unsupported e
scape sequence near index 3
D:\hadoop-trunk\hadoop\hadoop-common-project\hadoop-common\src\main\webapps\static
{noformat}

Following added as part of this jira,

{code}

resource-gz
generate-resources

  resource-gz


  
${basedir}/src/main/webapps/static
  
${basedir}/target/webapps/static
  js,css

  
{code}


was (Author: brahmareddy):
Compilation fails with following error, after this checked-in.

 {noformat}
[ERROR] Failed to execute goal 
org.apache.hadoop:hadoop-maven-plugins:3.0.0-alpha2-SNAPSHOT:resource-gz 
(resource-gz) on
 project hadoop-common: org.apache.maven.plugin.MojoExecutionException: 
java.util.regex.PatternSyntaxException: Illegal/
unsupported escape sequence near index 3
[ERROR] 
D:\hadoop-trunk\hadoop\hadoop-common-project\hadoop-common\src\main\webapps\static
[ERROR] ^
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal 
org.apache.hadoop:hadoop-maven-plugins:3.
0.0-alpha2-SNAPSHOT:resource-gz (resource-gz) on project hadoop-common: 
org.apache.maven.plugin.MojoExecutionException:
java.util.regex.PatternSyntaxException:

[jira] [Comment Edited] (HADOOP-10075) Update jetty dependency to version 9

2016-10-24 Thread John Zhuge (JIRA) 

[ 
https://issues.apache.org/jira/browse/HADOOP-10075?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15602216#comment-15602216
 ] 

John Zhuge edited comment on HADOOP-10075 at 10/24/16 3:14 PM:
---

Trailing white spaces in patch 010 at line 5624, 5631-5633, 8125, 8131, 
8159-8160, 8173-8175, 8178, and many others.



was (Author: jzhuge):
Trailing white spaces in patch 010 at line 5624, 5631, 5632, 5633, and 8125.


> Update jetty dependency to version 9
> 
>
> Key: HADOOP-10075
> URL: https://issues.apache.org/jira/browse/HADOOP-10075
> Project: Hadoop Common
>  Issue Type: Improvement
>Affects Versions: 2.2.0, 2.6.0
>Reporter: Robert Rati
>Assignee: Robert Kanter
>Priority: Critical
> Attachments: HADOOP-10075-002-wip.patch, HADOOP-10075.003.patch, 
> HADOOP-10075.004.patch, HADOOP-10075.005.patch, HADOOP-10075.006.patch, 
> HADOOP-10075.007.patch, HADOOP-10075.008.patch, HADOOP-10075.009.patch, 
> HADOOP-10075.010.patch, HADOOP-10075.patch
>
>
> Jetty6 is no longer maintained.  Update the dependency to jetty9.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

[jira] [Comment Edited] (HADOOP-10075) Update jetty dependency to version 9

2016-09-20 Thread Robert Kanter (JIRA) 

[ 
https://issues.apache.org/jira/browse/HADOOP-10075?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15507314#comment-15507314
 ] 

Robert Kanter edited comment on HADOOP-10075 at 9/20/16 6:11 PM:
-

The 004 patch does the following:
- Fixed unit tests
- Fixed deprecated warnings
- Removed some missed Jetty 6 usage
- Removed {{js.gz}} and {{css.gz}} files, so only the original files are in the 
source codebase.  A [maven plugin|https://github.com/phuonghuynh/compressor] is 
used to gzip these files into the target directories during build time.  We now 
do this to all {{js}} and {{css} files, unlike before where we had some that 
were gzipped and some that were not.

Note that I used {{--binary}} when generating the patch because it deletes some 
gzip files.


was (Author: rkanter):
The 004 patch does the following:
- Fixed unit tests
- Fixed deprecated warnings
- Removed some missed Jetty 6 usage
- Removed {{js.gz}} and {{css.gz}} files, so only the original files are in the 
source codebase.  A [maven plugin|https://github.com/phuonghuynh/compressor] is 
used to gzip these files into the target directories during build time.

Note that I used {{--binary}} when generating the patch because it deletes some 
gzip files.

> Update jetty dependency to version 9
> 
>
> Key: HADOOP-10075
> URL: https://issues.apache.org/jira/browse/HADOOP-10075
> Project: Hadoop Common
>  Issue Type: Improvement
>Affects Versions: 2.2.0, 2.6.0
>Reporter: Robert Rati
>Assignee: Robert Kanter
> Attachments: HADOOP-10075-002-wip.patch, HADOOP-10075.003.patch, 
> HADOOP-10075.004.patch, HADOOP-10075.patch
>
>
> Jetty6 is no longer maintained.  Update the dependency to jetty9.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

[jira] [Comment Edited] (HADOOP-10075) Update jetty dependency to version 9

2016-09-07 Thread Tsuyoshi Ozawa (JIRA) 

[ 
https://issues.apache.org/jira/browse/HADOOP-10075?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15472876#comment-15472876
 ] 

Tsuyoshi Ozawa edited comment on HADOOP-10075 at 9/8/16 5:53 AM:
-

[~rkanter] I looked over the result of  grep -r jetty --include="*.java": it's 
used only in server-side excluding tests, so we can upgrade jetty with minimum 
pain on branch-3 :-)


was (Author: ozawa):
[~rkanter] I looked over the result of  grep -r jetty --include="*.java": it's 
used only server-side, so we can upgrade jetty safely :-)

> Update jetty dependency to version 9
> 
>
> Key: HADOOP-10075
> URL: https://issues.apache.org/jira/browse/HADOOP-10075
> Project: Hadoop Common
>  Issue Type: Improvement
>Affects Versions: 2.2.0, 2.6.0
>Reporter: Robert Rati
>Assignee: Robert Kanter
> Attachments: HADOOP-10075-002-wip.patch, HADOOP-10075.patch
>
>
> Jetty6 is no longer maintained.  Update the dependency to jetty9.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org