aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Zameer Manji" <zma...@apache.org>
Subject Re: Review Request 41392: Upgrade to jetty 9.3.6.v20151106.
Date Tue, 15 Dec 2015 19:46:44 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41392/#review110552
-----------------------------------------------------------



src/main/java/org/apache/aurora/scheduler/http/HttpStatsFilter.java (line 57)
<https://reviews.apache.org/r/41392/#comment170487>

    Drive by post-commit comment. Can we not supress deprecation without a comment explaining
what it is supressing?



src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java (line 377)
<https://reviews.apache.org/r/41392/#comment170488>

    The addition of Slf4J here further complicates our logging story. Plus, I'm not aware
of us bundling any slf4j backends with our build so I am pretty sure these request messages
are now lost.
    
    Bill, can you confirm/deny these messages are kept and if so, what is is the slf4j backend
that is used?


- Zameer Manji


On Dec. 14, 2015, 10:39 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41392/
> -----------------------------------------------------------
> 
> (Updated Dec. 14, 2015, 10:39 p.m.)
> 
> 
> Review request for Aurora and Maxim Khutornenko.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Jetty 7 and 8 are are at EOL and in maintenace mode.  This brings us up to the latest.
> 
> 
> Diffs
> -----
> 
>   build.gradle ab1ec1ccebc3366db9531d6518507c7e7a5a6e64 
>   src/main/java/org/apache/aurora/scheduler/http/HttpStatsFilter.java 7ffd10573b31089c850802854b344433d64ed08b

>   src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java c78315da4b7a23fac274e10e2c8bcb4001a0d94c

>   src/main/java/org/apache/aurora/scheduler/http/RequestLogger.java ef6a84879a2a90d2c7ff2a81a723b20c56d550c2

>   src/main/java/org/apache/aurora/scheduler/http/api/ApiModule.java 791fd45646f4b353122e723b43b22b44baf5279b

>   src/test/java/org/apache/aurora/scheduler/http/RequestLoggerTest.java a2f786a588633e82890c2e8d695a5be2fde45452

>   src/test/java/org/apache/aurora/scheduler/http/ServletFilterTest.java 3a18e94e3459b020f86a4f50e4f7e745ed33e670

>   src/test/sh/org/apache/aurora/e2e/test_kerberos_end_to_end.sh 82ff8682d9b22236f3488cd7c107e2bba87c1e23

> 
> Diff: https://reviews.apache.org/r/41392/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Bill Farner
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message