hello

2017-01-09 Thread Josh Soref
Hi,
I have a series of spelling fixes for Apache Tomcat [1].

If someone could suggest a preferred way to receive them, I'm happy to
split them up.

Some changes are more interesting than others (the change to KEYS is
interesting).

[1] https://github.com/jsoref/tomcat/commits/spelling

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: hello

2017-01-09 Thread Violeta Georgieva
Hi,

2017-01-09 10:51 GMT+02:00 Josh Soref :
>
> Hi,
> I have a series of spelling fixes for Apache Tomcat [1].
>
> If someone could suggest a preferred way to receive them, I'm happy to
> split them up.
>
> Some changes are more interesting than others (the change to KEYS is
> interesting).
>
> [1] https://github.com/jsoref/tomcat/commits/spelling

I'll take a look.

Regards,
Violeta

> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


Re: hello

2017-01-09 Thread Mark Thomas
On 09/01/2017 08:51, Josh Soref wrote:
> Hi,
> I have a series of spelling fixes for Apache Tomcat [1].
> 
> If someone could suggest a preferred way to receive them, I'm happy to
> split them up.

Pull request or patch file attached to a new Bugzilla issue is fine.

Mark


> 
> Some changes are more interesting than others (the change to KEYS is
> interesting).
> 
> [1] https://github.com/jsoref/tomcat/commits/spelling
> 
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
> 


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: hello

2017-01-09 Thread Violeta Georgieva
Hi Josh,

2017-01-09 11:11 GMT+02:00 Mark Thomas :
>
> On 09/01/2017 08:51, Josh Soref wrote:
> > Hi,
> > I have a series of spelling fixes for Apache Tomcat [1].
> >
> > If someone could suggest a preferred way to receive them, I'm happy to
> > split them up.
>
> Pull request or patch file attached to a new Bugzilla issue is fine.

I saw that you have separate changes for every spelling fix.
May be it is better to squash them and then make PR.

Regards,
Violeta

> Mark
>
>
> >
> > Some changes are more interesting than others (the change to KEYS is
> > interesting).
> >
> > [1] https://github.com/jsoref/tomcat/commits/spelling
> >
> > -
> > To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> > For additional commands, e-mail: dev-h...@tomcat.apache.org
> >
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


Re: hello

2017-01-09 Thread Josh Soref
I can certainly squash. The reason I personally retain splits by misspelled 
word is that it's easier to ‎rebase / resolve conflicts when I can see what 
word was misspelled.

Typically large projects prefer to have my changes split into a couple of 
patches/PRs instead of as a single commit.

As a quick review, you can do diff -r trunk -r spelling or similar to see the 
world. 
‎
[1] https://github.com/xbmc/xbmc/pull/11346

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: hello

2017-01-09 Thread Violeta Georgieva
Mark,

2017-01-09 11:11 GMT+02:00 Mark Thomas :
>
> On 09/01/2017 08:51, Josh Soref wrote:
> > Hi,
> > I have a series of spelling fixes for Apache Tomcat [1].
> >
> > If someone could suggest a preferred way to receive them, I'm happy to
> > split them up.
>
> Pull request or patch file attached to a new Bugzilla issue is fine.

Some of the fixes are in DBCP2 code [1]
Should we fix our copy of DBCP2? I think it's better this to be changed
directly in DBCP2 code.

[1]
https://github.com/jsoref/tomcat/commit/6c686154f8a780733bc14546056ef588c34d20b9

> Mark
>
>
> >
> > Some changes are more interesting than others (the change to KEYS is
> > interesting).
> >
> > [1] https://github.com/jsoref/tomcat/commits/spelling
> >
> > -
> > To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> > For additional commands, e-mail: dev-h...@tomcat.apache.org
> >
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


Re: hello

2017-01-09 Thread Mark Thomas
On 09/01/2017 09:35, Violeta Georgieva wrote:
> Mark,
> 
> 2017-01-09 11:11 GMT+02:00 Mark Thomas :
>>
>> On 09/01/2017 08:51, Josh Soref wrote:
>>> Hi,
>>> I have a series of spelling fixes for Apache Tomcat [1].
>>>
>>> If someone could suggest a preferred way to receive them, I'm happy to
>>> split them up.
>>
>> Pull request or patch file attached to a new Bugzilla issue is fine.
> 
> Some of the fixes are in DBCP2 code [1]
> Should we fix our copy of DBCP2? I think it's better this to be changed
> directly in DBCP2 code.

I agree. DBCP2 fixes should be applied to Commons and then we'll pick
them up when up next update.

Mark

> 
> [1]
> https://github.com/jsoref/tomcat/commit/6c686154f8a780733bc14546056ef588c34d20b9
> 
>> Mark
>>
>>
>>>
>>> Some changes are more interesting than others (the change to KEYS is
>>> interesting).
>>>
>>> [1] https://github.com/jsoref/tomcat/commits/spelling
>>>
>>> -
>>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>>> For additional commands, e-mail: dev-h...@tomcat.apache.org
>>>
>>
>>
>> -
>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>> For additional commands, e-mail: dev-h...@tomcat.apache.org
>>
> 


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: hello

2017-01-09 Thread Violeta Georgieva
Josh,

2017-01-09 11:38 GMT+02:00 Mark Thomas :
>
> On 09/01/2017 09:35, Violeta Georgieva wrote:
> > Mark,
> >
> > 2017-01-09 11:11 GMT+02:00 Mark Thomas :
> >>
> >> On 09/01/2017 08:51, Josh Soref wrote:
> >>> Hi,
> >>> I have a series of spelling fixes for Apache Tomcat [1].
> >>>
> >>> If someone could suggest a preferred way to receive them, I'm happy to
> >>> split them up.
> >>
> >> Pull request or patch file attached to a new Bugzilla issue is fine.
> >
> > Some of the fixes are in DBCP2 code [1]
> > Should we fix our copy of DBCP2? I think it's better this to be changed
> > directly in DBCP2 code.
>
> I agree. DBCP2 fixes should be applied to Commons and then we'll pick
> them up when up next update.

Please review this document
https://github.com/apache/tomcat/blob/trunk/MERGE.txt

We reuse code from other Apache projects

Violeta

> Mark
>
> >
> > [1]
> >
https://github.com/jsoref/tomcat/commit/6c686154f8a780733bc14546056ef588c34d20b9
> >
> >> Mark
> >>
> >>
> >>>
> >>> Some changes are more interesting than others (the change to KEYS is
> >>> interesting).
> >>>
> >>> [1] https://github.com/jsoref/tomcat/commits/spelling
> >>>
> >>> -
> >>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> >>> For additional commands, e-mail: dev-h...@tomcat.apache.org
> >>>
> >>
> >>
> >> -
> >> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> >> For additional commands, e-mail: dev-h...@tomcat.apache.org
> >>
> >
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


Re: [VOTE] Release Apache Tomcat 8.5.10

2017-01-09 Thread Mark Thomas
On 06/01/2017 15:58, Rémy Maucherat wrote:
> 2017-01-06 16:52 GMT+01:00 Violeta Georgieva :
> 
>> With r1775598 changes, the declaration of the scripting variable now is in
>> try/finally block and it is not visible outside this block.
>>
>> http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/a
>> pache/jasper/compiler/Generator.java?view=markup&pathrev=1775598#l2713
>>
> This is independent of the previous issue, so you can either remove the
> try/finally for simple tags, or fix it if you have a test on hand,

I'm thinking that these couple of Jasper issues plus the APR/native
issue I stumbled across make it worth cancelling the current 8.5.x and
9.0.x releases and re-rolling.

Thoughts?

Assuming these is general agreement, I'll probably re-tag later this
afternoon.

Mark


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



TomcatCon @ ApacheCon

2017-01-09 Thread Mark Thomas
All,

After speaking to various folks at various times about some form of
Tomcat conference, I've finally got around to actually doing something
about it. I'm like to try and do something alongside ApacheCon NA 2017.

I've started a discussion on the users list.

All help, support, ideas etc. welcome on that thread.

Cheers,

Mark

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: hello

2017-01-09 Thread Violeta Georgieva
2017-01-09 10:51 GMT+02:00 Josh Soref :
>
> Hi,
> I have a series of spelling fixes for Apache Tomcat [1].
>

I reviewed these changes. Please see the comments.
When you are ready squash the commits and make PR. I'll commit the changes.

> If someone could suggest a preferred way to receive them, I'm happy to
> split them up.
>
> Some changes are more interesting than others (the change to KEYS is
> interesting).

I didn't see any changes to KEYS.

>
> [1] https://github.com/jsoref/tomcat/commits/spelling
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


Re: hello

2017-01-09 Thread Violeta Georgieva
2017-01-09 11:23 GMT+02:00 Josh Soref :
>
> I can certainly squash. The reason I personally retain splits by
misspelled word is that it's easier to ‎rebase / resolve conflicts when I
can see what word was misspelled.
>
> Typically large projects prefer to have my changes split into a couple of
patches/PRs instead of as a single commit.
>

Yes that's true. I also prefer separate patches.
But in this case this is just a spelling correction and I think we can use
a single commit.

> As a quick review, you can do diff -r trunk -r spelling or similar to see
the world.
> ‎
> [1] https://github.com/xbmc/xbmc/pull/11346
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


Re: hello

2017-01-09 Thread Josh Soref
‎https://github.com/jsoref/tomcat/commit/7c6bc40ac289ee7e7c9655c69a529f18afb5b92f



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: [VOTE] Release Apache Tomcat 8.5.10

2017-01-09 Thread Violeta Georgieva
2017-01-09 13:55 GMT+02:00 Mark Thomas :
>
> On 06/01/2017 15:58, Rémy Maucherat wrote:
> > 2017-01-06 16:52 GMT+01:00 Violeta Georgieva :
> >
> >> With r1775598 changes, the declaration of the scripting variable now
is in
> >> try/finally block and it is not visible outside this block.
> >>
> >> http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/a
> >> pache/jasper/compiler/Generator.java?view=markup&pathrev=1775598#l2713
> >>
> > This is independent of the previous issue, so you can either remove the
> > try/finally for simple tags, or fix it if you have a test on hand,
>
> I'm thinking that these couple of Jasper issues plus the APR/native
> issue I stumbled across make it worth cancelling the current 8.5.x and
> 9.0.x releases and re-rolling.

+1

>
> Thoughts?
>
> Assuming these is general agreement, I'll probably re-tag later this
> afternoon.
>
> Mark
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


Re: hello

2017-01-09 Thread Violeta Georgieva
2017-01-09 14:23 GMT+02:00 Josh Soref :
>
> ‎
https://github.com/jsoref/tomcat/commit/7c6bc40ac289ee7e7c9655c69a529f18afb5b92f

Let's not touch the KEYS file

>
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


Re: hello

2017-01-09 Thread Josh Soref
I can drop that, but please note that the spelling is listed as corrected in 
the MIT key server. I really wonder how tomcat managed to corrupt it.



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: TomcatCon @ ApacheCon

2017-01-09 Thread jean-frederic clere
On 01/09/2017 12:58 PM, Mark Thomas wrote:
> All,
> 
> After speaking to various folks at various times about some form of
> Tomcat conference, I've finally got around to actually doing something
> about it. I'm like to try and do something alongside ApacheCon NA 2017.

That makes sense since the ApacheCon I have done 2 JUGs on tomcat-8.5,
got a bunch of questions and requests to do more of the same.

Cheers

Jean-Frederic

> 
> I've started a discussion on the users list.
> 
> All help, support, ideas etc. welcome on that thread.
> 
> Cheers,
> 
> Mark
> 
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
> 
> 


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: hello

2017-01-09 Thread Violeta Georgieva
Jean-Frederic,

2017-01-09 14:23 GMT+02:00 Josh Soref :
>
> ‎
https://github.com/jsoref/tomcat/commit/7c6bc40ac289ee7e7c9655c69a529f18afb5b92f

Can you take a look at this change? It's about a spelling mistake in your
email address in the KEYS file.

Thanks a lot,
Violeta

>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>


svn commit: r1777967 - in /tomcat/trunk: java/org/apache/tomcat/util/net/AprEndpoint.java java/org/apache/tomcat/util/net/LocalStrings.properties webapps/docs/changelog.xml

2017-01-09 Thread markt
Author: markt
Date: Mon Jan  9 14:12:36 2017
New Revision: 1777967

URL: http://svn.apache.org/viewvc?rev=1777967&view=rev
Log:
Remove problematic work-around

Modified:
tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
tomcat/trunk/java/org/apache/tomcat/util/net/LocalStrings.properties
tomcat/trunk/webapps/docs/changelog.xml

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java?rev=1777967&r1=1777966&r2=1777967&view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Mon Jan  9 
14:12:36 2017
@@ -2362,14 +2362,6 @@ public class AprEndpoint extends Abstrac
 return result;
 } else if (result == 0 || -result == Status.EAGAIN) {
 return 0;
-} else if (-result == Status.APR_EGENERAL && isSecure()) {
-// Not entirely sure why this is necessary. Testing to date 
has not
-// identified any issues with this but log it so it can be 
tracked
-// if it is suspected of causing issues in the future.
-if (log.isDebugEnabled()) {
-log.debug(sm.getString("socket.apr.read.sslGeneralError", 
getSocket(), this));
-}
-return 0;
 } else if ((-result) == Status.ETIMEDOUT || (-result) == 
Status.TIMEUP) {
 if (block) {
 throw new 
SocketTimeoutException(sm.getString("iib.readtimeout"));

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/LocalStrings.properties?rev=1777967&r1=1777966&r2=1777967&view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/util/net/LocalStrings.properties 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/LocalStrings.properties Mon 
Jan  9 14:12:36 2017
@@ -115,7 +115,6 @@ socket.sslreneg=Exception re-negotiating
 
 socket.apr.clientAbort=The client aborted the connection.
 socket.apr.read.error=Unexpected error [{0}] reading data from the APR/native 
socket [{1}] with wrapper [{2}].
-socket.apr.read.sslGeneralError=An APR general error was returned by the SSL 
read operation on APR/native socket [{0}] with wrapper [{1}]. It will be 
treated as EAGAIN and the socket returned to the poller.
 socket.apr.write.error=Unexpected error [{0}] writing data to the APR/native 
socket [{1}] with wrapper [{2}].
 socket.apr.closed=The socket [{0}] associated with this connection has been 
closed.
 

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1777967&r1=1777966&r2=1777967&view=diff
==
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Mon Jan  9 14:12:36 2017
@@ -51,6 +51,12 @@
 Ensure UpgradeProcessor instances associated with closed connections 
are
 removed from the map of current connections to Processors. (markt)
   
+  
+Remove a workaround for a problem previously reported with WebSocket,
+TLS and APR that treated some error conditions as not errors. The
+original problem cannot be reproduced with the current code and the
+work-around is now causing problems. (markt)
+  
 
   
   



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1777968 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java java/org/apache/tomcat/util/net/LocalStrings.properties webapps/docs/changelog.xml

2017-01-09 Thread markt
Author: markt
Date: Mon Jan  9 14:13:19 2017
New Revision: 1777968

URL: http://svn.apache.org/viewvc?rev=1777968&view=rev
Log:
Remove problematic work-around

Modified:
tomcat/tc8.5.x/trunk/   (props changed)
tomcat/tc8.5.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
tomcat/tc8.5.x/trunk/java/org/apache/tomcat/util/net/LocalStrings.properties
tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc8.5.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jan  9 14:13:19 2017
@@ -1 +1 @@
-/tomcat/trunk:1734785,1734799,1734845,1734928,1735041,1735044,1735480,1735577,1735597,1735599-1735600,1735615,1736145,1736162,1736209,1736280,1736297,1736299,1736489,1736646,1736703,1736836,1736849,1737104-1737105,1737112,1737117,1737119-1737120,1737155,1737157,1737192,1737280,1737339,1737632,1737664,1737715,1737748,1737785,1737834,1737860,1737903,1737959,1738005,1738007,1738014-1738015,1738018,1738022,1738039,1738043,1738059-1738060,1738147,1738149,1738174-1738175,1738261,1738589,1738623-1738625,1738643,1738816,1738850,1738855,1738946-1738948,1738953-1738954,1738979,1738982,1739079-1739081,1739087,1739113,1739153,1739172,1739176,1739191,1739474,1739726,1739762,1739775,1739814,1739817-1739818,1739975,1740131,1740324,1740465,1740495,1740508-1740509,1740520,1740535,1740707,1740803,1740810,1740969,1740980,1740991,1740997,1741015,1741033,1741036,1741058,1741060,1741080,1741147,1741159,1741164,1741173,1741181,1741190,1741197,1741202,1741208,1741213,1741221,1741225,1741232,1741409,1741501
 
,1741677,1741892,1741896,1741984,1742023,1742042,1742071,1742090,1742093,1742101,1742105,1742111,1742139,1742146,1742148,1742166,1742181,1742184,1742187,1742246,1742248-1742251,1742263-1742264,1742268,1742276,1742369,1742387,1742448,1742509-1742512,1742917,1742919,1742933,1742975-1742976,1742984,1742986,1743019,1743115,1743117,1743124-1743125,1743134,1743425,1743554,1743679,1743696-1743698,1743700-1743701,1744058,1744064-1744065,1744125,1744194,1744229,1744270,1744323,1744432,1744684,1744697,1744705,1744713,1744760,1744786,1745083,1745142-1745143,1745145,1745177,1745179-1745180,1745227,1745248,1745254,1745337,1745467,1745473,1745576,1745735,1745744,1746304,1746306-1746307,1746319,1746327,1746338,1746340-1746341,1746344,1746427,1746441,1746473,1746490,1746492,1746495-1746496,1746499-1746501,1746503-1746507,1746509,1746549,1746551,1746554,1746556,1746558,1746584,1746620,1746649,1746724,1746939,1746989,1747014,1747028,1747035,1747210,1747225,1747234,1747253,1747404,1747506,1747536,1747
 
924,1747980,1747993,1748001,1748253,1748452,1748547,1748629,1748676,1748715,1749287,1749296,1749328,1749373,1749465,1749506,1749508,1749665-1749666,1749763,1749865-1749866,1749898,1749978,1749980,1750011,1750015,1750056,1750480,1750617,1750634,1750692,1750697,1750700,1750703,1750707,1750714,1750718,1750723,1750774,1750899,1750975,1750995,1751061,1751097,1751173,1751438,1751447,1751463,1751702,1752212,1752737,1752745,1753078,1753080,1753358,1753363,1754111,1754140-1754141,1754281,1754310,1754445,1754467,1754494,1754496,1754528,1754532-1754533,1754613,1754714,1754874,1754941,1754944,1754950-1754951,1755005,1755007,1755009,1755132,1755180-1755181,1755185,1755190,1755204-1755206,1755208,1755214,1755224,1755227,1755230,1755629,1755646-1755647,1755650,1755653,1755675,1755680,1755683,1755693,1755717,1755731-1755737,1755812,1755828,1755884,1755890,1755918-1755919,1755942,1755958,1755960,1755970,1755993,1756013,1756019,1756039,1756056,1756083-1756114,1756175,1756288-1756289,1756408-1756410,1
 
756778,1756798,1756878,1756898,1756939,1757123-1757124,1757126,1757128,1757132-1757133,1757136,1757145,1757167-1757168,1757175,1757180,1757182,1757195,1757271,1757278,1757347,1757353-1757354,1757363,1757374,1757399,1757406,1757408,1757485,1757495,1757499,1757527,1757578,1757684,1757722,1757727,1757790,1757799,1757813,1757853,1757883,1757903,1757976,1757997,1758000,1758058,1758072-1758075,1758078-1758079,1758223,1758257,1758261,1758276,1758292,1758369,1758378-1758383,1758421,1758423,1758425-1758427,1758430,1758443,1758448,1758459,1758483,1758486-1758487,1758499,1758525,1758556,1758580,1758582,1758584,1758588,1758842,1759019,1759212,1759224,1759227,1759252,1759274,1759513-1759516,1759611,1759757,1759785-1759790,1760005,1760022,1760109-1760110,1760135,1760200-1760201,1760227,1760300,1760397,1760446,1760454,1760640,1760648,1761057,1761422,1761491,1761498,1761500-1761501,1761550,1761553,1761572,1761574,1761625-1761626,1761628,1761682,1761740,1761752,1762051-1762053,1762123,1762168,176217
 
2,1762182,1762201-1762202,1762204,1762208,1762288,1762296,1762324,1762348,1762353,1762362,1762374,1762492,1762503,1762505,1762541,1762608,1762710,1762753,1762766,1762769,1762944,1762947,1762953,1763167,1763179,1763232,1763259,1763271-1763272,1763276-1763277,1763319-1763320,1763370,1763372,1763375,1763377,1763393,1763412,1763430,1763450,1763462,1763505,1763511-1763512,1763516,1

[Bug 60461] SIGSEGV in SSLSocket.getInfos

2017-01-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60461

Ludovic Pénet  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEEDINFO|RESOLVED

--- Comment #8 from Ludovic Pénet  ---
Hi.

I tested with Tomcat 8.5.10 (built from sources as the release is still in
progress), and it works fine.

So, I think we can close this issue.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Question about Windows zip distro

2017-01-09 Thread Coty Sutherland
Hi all,

Do we intentionally ship the shell scripts along with the
tomcat-native and commons-daemon-native tarballs in bin for the
Windows distribution?

+++
$ ll apache-tomcat-7.0.73-windows-x64.zip/apache-tomcat-7.0.73/bin/ |
egrep '(tar.gz|sh)$'
-rw-r--r--. 1 coty coty   21646 Nov  7 23:29 catalina.sh
-rw-r--r--. 1 coty coty  204944 Nov  7 23:29 commons-daemon-native.tar.gz
-rw-r--r--. 1 coty coty1922 Nov  7 23:29 configtest.sh
-rw-r--r--. 1 coty coty7888 Nov  7 23:29 daemon.sh
-rw-r--r--. 1 coty coty1965 Nov  7 23:29 digest.sh
-rw-r--r--. 1 coty coty3547 Nov  7 23:29 setclasspath.sh
-rw-r--r--. 1 coty coty1902 Nov  7 23:29 shutdown.sh
-rw-r--r--. 1 coty coty1904 Nov  7 23:29 startup.sh
-rw-r--r--. 1 coty coty  391853 Nov  7 23:29 tomcat-native.tar.gz
-rw-r--r--. 1 coty coty5024 Nov  7 23:29 tool-wrapper.sh
-rw-r--r--. 1 coty coty1908 Nov  7 23:29 version.sh
+++

If not, I can amend the build.xml so that it doesn't include them; I
just thought it looks odd (and would save us 631.9K off the zip size).



Thanks,
Coty

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Wrong type of the value of javax.servlet.error.exception_type

2017-01-09 Thread Martin Grigorov
Hi Chris,

On Fri, Jan 6, 2017 at 9:49 PM, Christopher Schultz <
ch...@christopherschultz.net> wrote:

> Martin,
>
> On 1/6/17 8:23 AM, Martin Grigorov wrote:
> > I've just upgraded my local/development Tomcat to 8.5.10 and I've hit a
> > problem in Apache Wicket:
> >
> > java.lang.ClassCastException: java.lang.String cannot be cast to
> > java.lang.Class
> > at
> > org.apache.wicket.protocol.http.servlet.ErrorAttributes.
> of(ErrorAttributes.java:152)
> >
> >
> > line 152 is:
> > Class type = (Class > Throwable>)request.getAttribute("javax.servlet.error.exception_type");
> >
> > In a debugger I see that the value is the class name, not the class
> itself.
> >
> > According to
> > http://download.oracle.com/otn-pub/jcp/servlet-3.0-fr-
> eval-oth-JSpec/servlet-3_0-final-spec.pdf,
> > p.10.9.1, table 10-1 the type of the value should be:
> >
> > Request Attributes  Type
> > javax.servlet.error.status_code java.lang.Integer
> > javax.servlet.error.exception_type java.lang.Class   <<< a Class
> > javax.servlet.error.message java.lang.String
> > javax.servlet.error.exception java.lang.Throwable
> > javax.servlet.error.request_uri java.lang.String
> > javax.servlet.error.servlet_name java.lang.String
> >
> >
> > I didn't see any change related to this between 8.5.8 and 8.5.10 so I'm
> not
> > sure whether this should be a stopper for the release of 8.5.10.
>
> I only see a single instance of javax.servlet.error.exception_type in
> 8.5 trunk (in javax/servlet/RequestDispatcher.ERROR_EXCEPTION_TYPE) and
> a single use of ERROR_EXCEPTION_TYPE itself (in
> o.a.c.core.StandardHostValve) and the code is fairly straightforward:
>
> // line 330:
> request.setAttribute(RequestDispatcher.ERROR_EXCEPTION_TYPE,
>   realError.getClass());
>
> The reference "realError" is a java.lang.Throwable, and, presumably,
> realError.getClass will always return a java.lang.Class object.
>
> Can you see where that request attribute is being set in your test? I
> wonder if either Wicket or the application itself is abusing that
> request attribute.
>

Thanks for your answer!
Since Tomcat is not guilty here it must be the middleware! I've found it
easily:
https://github.com/spring-projects/spring-boot/search?utf8=%E2%9C%93&q=ERROR_EXCEPTION_TYPE
I'll log an issue at Spring Boot!



>
> -chris
>
>


Memory leak found in java.util.concurrent.ForkJoinPool

2017-01-09 Thread Mark Thomas
All,

As a result of a thread on the users list, a memory leak has been
discovered in java.util.concurrent.ForkJoinPool.commonPool. It appears
that use of this pool is hard-coded in multiple places such as
Collection.parallelStream().forEach().

I have some ideas about how to work-around the leaks with
JreMemoryLeakPreventionListener but any code that depends on the Tomcat
class loader being set (e.g. JNDI lookup) is going to break if the user
tries to use Collection.parallelStream().


Rory,

The review Id for my report is 9046841. Anything you can do to get this
accepted and in front of the right folks to fix it would be appreciated.

Kind regards,

Mark

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Question about Windows zip distro

2017-01-09 Thread Mark Thomas
On 09/01/2017 18:24, Coty Sutherland wrote:
> Hi all,
> 
> Do we intentionally ship the shell scripts along with the
> tomcat-native and commons-daemon-native tarballs in bin for the
> Windows distribution?

Yes.

The only variation is the difference in line-endings.

I believe the intention is to provide users with as much flexibility as
possible (i.e. you can use the .zip on Linux if you really want to)
while making it work out of the box for the more obvious/popular use cases.

Mark


> 
> +++
> $ ll apache-tomcat-7.0.73-windows-x64.zip/apache-tomcat-7.0.73/bin/ |
> egrep '(tar.gz|sh)$'
> -rw-r--r--. 1 coty coty   21646 Nov  7 23:29 catalina.sh
> -rw-r--r--. 1 coty coty  204944 Nov  7 23:29 commons-daemon-native.tar.gz
> -rw-r--r--. 1 coty coty1922 Nov  7 23:29 configtest.sh
> -rw-r--r--. 1 coty coty7888 Nov  7 23:29 daemon.sh
> -rw-r--r--. 1 coty coty1965 Nov  7 23:29 digest.sh
> -rw-r--r--. 1 coty coty3547 Nov  7 23:29 setclasspath.sh
> -rw-r--r--. 1 coty coty1902 Nov  7 23:29 shutdown.sh
> -rw-r--r--. 1 coty coty1904 Nov  7 23:29 startup.sh
> -rw-r--r--. 1 coty coty  391853 Nov  7 23:29 tomcat-native.tar.gz
> -rw-r--r--. 1 coty coty5024 Nov  7 23:29 tool-wrapper.sh
> -rw-r--r--. 1 coty coty1908 Nov  7 23:29 version.sh
> +++
> 
> If not, I can amend the build.xml so that it doesn't include them; I
> just thought it looks odd (and would save us 631.9K off the zip size).
> 
> 
> 
> Thanks,
> Coty
> 
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
> 


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r1778061 - in /tomcat/trunk: build.xml java/org/apache/catalina/core/JreMemoryLeakPreventionListener.java java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java webapps/docs

2017-01-09 Thread markt
Author: markt
Date: Mon Jan  9 21:50:53 2017
New Revision: 1778061

URL: http://svn.apache.org/viewvc?rev=1778061&view=rev
Log:
Add memory protection for ForkJoinPool.commonPool() related memory leaks.

Added:

tomcat/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java
   (with props)
Modified:
tomcat/trunk/build.xml

tomcat/trunk/java/org/apache/catalina/core/JreMemoryLeakPreventionListener.java
tomcat/trunk/webapps/docs/changelog.xml
tomcat/trunk/webapps/docs/config/listeners.xml

Modified: tomcat/trunk/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/build.xml?rev=1778061&r1=1778060&r2=1778061&view=diff
==
--- tomcat/trunk/build.xml (original)
+++ tomcat/trunk/build.xml Mon Jan  9 21:50:53 2017
@@ -336,6 +336,8 @@
 
 
 
+
+
 
 
 

Modified: 
tomcat/trunk/java/org/apache/catalina/core/JreMemoryLeakPreventionListener.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/JreMemoryLeakPreventionListener.java?rev=1778061&r1=1778060&r2=1778061&view=diff
==
--- 
tomcat/trunk/java/org/apache/catalina/core/JreMemoryLeakPreventionListener.java 
(original)
+++ 
tomcat/trunk/java/org/apache/catalina/core/JreMemoryLeakPreventionListener.java 
Mon Jan  9 21:50:53 2017
@@ -25,6 +25,7 @@ import java.net.URL;
 import java.net.URLConnection;
 import java.sql.DriverManager;
 import java.util.StringTokenizer;
+import java.util.concurrent.ForkJoinPool;
 
 import javax.xml.parsers.DocumentBuilder;
 import javax.xml.parsers.DocumentBuilderFactory;
@@ -33,6 +34,7 @@ import javax.xml.parsers.ParserConfigura
 import org.apache.catalina.Lifecycle;
 import org.apache.catalina.LifecycleEvent;
 import org.apache.catalina.LifecycleListener;
+import org.apache.catalina.startup.SafeForkJoinWorkerThreadFactory;
 import org.apache.juli.logging.Log;
 import org.apache.juli.logging.LogFactory;
 import org.apache.tomcat.util.ExceptionUtils;
@@ -63,6 +65,8 @@ public class JreMemoryLeakPreventionList
 private static final StringManager sm =
 StringManager.getManager(Constants.Package);
 
+private static final String FORK_JOIN_POOL_THREAD_FACTORY_PROPERTY =
+"java.util.concurrent.ForkJoinPool.common.threadFactory";
 /**
  * Protect against the memory leak caused when the first call to
  * java.awt.Toolkit.getDefaultToolkit() is triggered
@@ -161,6 +165,19 @@ public class JreMemoryLeakPreventionList
 }
 
 /**
+ * {@link ForkJoinPool#commonPool()} creates a thread pool that, by 
default,
+ * creates threads that retain references to the thread context class
+ * loader.
+ */
+private boolean forkJoinCommonPoolProtection = true;
+public boolean getForkJoinCommonPoolProtection() {
+return forkJoinCommonPoolProtection;
+}
+public void setForkJoinCommonPoolProtection(boolean 
forkJoinCommonPoolProtection) {
+this.forkJoinCommonPoolProtection = forkJoinCommonPoolProtection;
+}
+
+/**
  * List of comma-separated fully qualified class names to load and 
initialize during
  * the startup of this Listener. This allows to pre-load classes that are 
known to
  * provoke classloader leaks if they are loaded during a request 
processing.
@@ -336,6 +353,17 @@ public class JreMemoryLeakPreventionList
 }
 }
 
+/*
+ * Present in Java 8 onwards
+ */
+if (forkJoinCommonPoolProtection) {
+// Don't override any explicitly set property
+if 
(System.getProperty(FORK_JOIN_POOL_THREAD_FACTORY_PROPERTY) == null) {
+
System.setProperty(FORK_JOIN_POOL_THREAD_FACTORY_PROPERTY,
+
SafeForkJoinWorkerThreadFactory.class.getName());
+}
+}
+
 if (classesToInitialize != null) {
 StringTokenizer strTok =
 new StringTokenizer(classesToInitialize, ", \r\n\t");

Added: 
tomcat/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java?rev=1778061&view=auto
==
--- 
tomcat/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java
 (added)
+++ 
tomcat/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java
 Mon Jan  9 21:50:53 2017
@@ -0,0 +1,46 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licens

svn commit: r1778065 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/catalina/core/ java/org/apache/catalina/startup/ webapps/docs/ webapps/docs/config/

2017-01-09 Thread markt
Author: markt
Date: Mon Jan  9 21:55:54 2017
New Revision: 1778065

URL: http://svn.apache.org/viewvc?rev=1778065&view=rev
Log:
Add memory protection for ForkJoinPool.commonPool() related memory leaks.

Added:

tomcat/tc8.5.x/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java
  - copied unchanged from r1778061, 
tomcat/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java
Modified:
tomcat/tc8.5.x/trunk/   (props changed)
tomcat/tc8.5.x/trunk/build.xml

tomcat/tc8.5.x/trunk/java/org/apache/catalina/core/JreMemoryLeakPreventionListener.java
tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml
tomcat/tc8.5.x/trunk/webapps/docs/config/listeners.xml

Propchange: tomcat/tc8.5.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jan  9 21:55:54 2017
@@ -1 +1 @@
-/tomcat/trunk:1734785,1734799,1734845,1734928,1735041,1735044,1735480,1735577,1735597,1735599-1735600,1735615,1736145,1736162,1736209,1736280,1736297,1736299,1736489,1736646,1736703,1736836,1736849,1737104-1737105,1737112,1737117,1737119-1737120,1737155,1737157,1737192,1737280,1737339,1737632,1737664,1737715,1737748,1737785,1737834,1737860,1737903,1737959,1738005,1738007,1738014-1738015,1738018,1738022,1738039,1738043,1738059-1738060,1738147,1738149,1738174-1738175,1738261,1738589,1738623-1738625,1738643,1738816,1738850,1738855,1738946-1738948,1738953-1738954,1738979,1738982,1739079-1739081,1739087,1739113,1739153,1739172,1739176,1739191,1739474,1739726,1739762,1739775,1739814,1739817-1739818,1739975,1740131,1740324,1740465,1740495,1740508-1740509,1740520,1740535,1740707,1740803,1740810,1740969,1740980,1740991,1740997,1741015,1741033,1741036,1741058,1741060,1741080,1741147,1741159,1741164,1741173,1741181,1741190,1741197,1741202,1741208,1741213,1741221,1741225,1741232,1741409,1741501
 
,1741677,1741892,1741896,1741984,1742023,1742042,1742071,1742090,1742093,1742101,1742105,1742111,1742139,1742146,1742148,1742166,1742181,1742184,1742187,1742246,1742248-1742251,1742263-1742264,1742268,1742276,1742369,1742387,1742448,1742509-1742512,1742917,1742919,1742933,1742975-1742976,1742984,1742986,1743019,1743115,1743117,1743124-1743125,1743134,1743425,1743554,1743679,1743696-1743698,1743700-1743701,1744058,1744064-1744065,1744125,1744194,1744229,1744270,1744323,1744432,1744684,1744697,1744705,1744713,1744760,1744786,1745083,1745142-1745143,1745145,1745177,1745179-1745180,1745227,1745248,1745254,1745337,1745467,1745473,1745576,1745735,1745744,1746304,1746306-1746307,1746319,1746327,1746338,1746340-1746341,1746344,1746427,1746441,1746473,1746490,1746492,1746495-1746496,1746499-1746501,1746503-1746507,1746509,1746549,1746551,1746554,1746556,1746558,1746584,1746620,1746649,1746724,1746939,1746989,1747014,1747028,1747035,1747210,1747225,1747234,1747253,1747404,1747506,1747536,1747
 
924,1747980,1747993,1748001,1748253,1748452,1748547,1748629,1748676,1748715,1749287,1749296,1749328,1749373,1749465,1749506,1749508,1749665-1749666,1749763,1749865-1749866,1749898,1749978,1749980,1750011,1750015,1750056,1750480,1750617,1750634,1750692,1750697,1750700,1750703,1750707,1750714,1750718,1750723,1750774,1750899,1750975,1750995,1751061,1751097,1751173,1751438,1751447,1751463,1751702,1752212,1752737,1752745,1753078,1753080,1753358,1753363,1754111,1754140-1754141,1754281,1754310,1754445,1754467,1754494,1754496,1754528,1754532-1754533,1754613,1754714,1754874,1754941,1754944,1754950-1754951,1755005,1755007,1755009,1755132,1755180-1755181,1755185,1755190,1755204-1755206,1755208,1755214,1755224,1755227,1755230,1755629,1755646-1755647,1755650,1755653,1755675,1755680,1755683,1755693,1755717,1755731-1755737,1755812,1755828,1755884,1755890,1755918-1755919,1755942,1755958,1755960,1755970,1755993,1756013,1756019,1756039,1756056,1756083-1756114,1756175,1756288-1756289,1756408-1756410,1
 
756778,1756798,1756878,1756898,1756939,1757123-1757124,1757126,1757128,1757132-1757133,1757136,1757145,1757167-1757168,1757175,1757180,1757182,1757195,1757271,1757278,1757347,1757353-1757354,1757363,1757374,1757399,1757406,1757408,1757485,1757495,1757499,1757527,1757578,1757684,1757722,1757727,1757790,1757799,1757813,1757853,1757883,1757903,1757976,1757997,1758000,1758058,1758072-1758075,1758078-1758079,1758223,1758257,1758261,1758276,1758292,1758369,1758378-1758383,1758421,1758423,1758425-1758427,1758430,1758443,1758448,1758459,1758483,1758486-1758487,1758499,1758525,1758556,1758580,1758582,1758584,1758588,1758842,1759019,1759212,1759224,1759227,1759252,1759274,1759513-1759516,1759611,1759757,1759785-1759790,1760005,1760022,1760109-1760110,1760135,1760200-1760201,1760227,1760300,1760397,1760446,1760454,1760640,1760648,1761057,1761422,1761491,1761498,1761500-1761501,1761550,1761553,1761572,1761574,1761625-1761626,1761628,1761682,1761740,1761752,1762051-1762053,1762123,1762168,176217
 
2,1762182,1762201-1762202,1762204,1762208,1762288,1762296,1762324,1762348,176

svn commit: r1778068 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/core/ java/org/apache/catalina/startup/ webapps/docs/ webapps/docs/config/

2017-01-09 Thread markt
Author: markt
Date: Mon Jan  9 22:19:48 2017
New Revision: 1778068

URL: http://svn.apache.org/viewvc?rev=1778068&view=rev
Log:
Add memory protection for ForkJoinPool.commonPool() related memory leaks.
The 7.0.x fix is a little more involved since the ThreadFactory required can 
only be compiled with Java 7 onwards.

Added:

tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java
  - copied unchanged from r1778065, 
tomcat/tc8.5.x/trunk/java/org/apache/catalina/startup/SafeForkJoinWorkerThreadFactory.java
Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/build.xml

tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/JreMemoryLeakPreventionListener.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
tomcat/tc7.0.x/trunk/webapps/docs/config/listeners.xml

Propchange: tomcat/tc7.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jan  9 22:19:48 2017
@@ -1,3 +1,3 @@
 
/tomcat/tc8.0.x/trunk:1636525,1637336,1637685,1637709,1638726,1640089,1640276,1640349,1640363,1640366,1640642,1640672,1640674,1640689,1640884,1641001,1641065,1641067,1641375,1641638,1641723,1641726,1641729-1641730,1641736,1641988,1642669-1642670,1642698,1642701,1643205,1643215,1643217,1643230,1643232,1643273,1643285,1643329-1643330,1643511,1643513,1643521,1643539,1643571,1643581-1643582,1643635,1643655,1643738,1643964,1644018,1644333,1644954,1644992,1645014,1645360,1645456,1645627,1645642,1645686,1645903-1645904,1645908-1645909,1645913,1645920,1646458,1646460-1646462,1646735,1646738-1646741,1646744,1646746,1646748-1646755,1646757,1646759-1646760,1647043,1648816,1651420-1651422,1651844,1652926,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1656592,1659907,1662986,1663265,1663278,1663325,1663535,1663567,1663679,1663997,1664175,1664321,1664872,1665061,1665086,1666027,1666395,1666503,1666506,1666560,1666570,1666581,1666759,1666967,1666988,1667553
 
-1667555,1667558,1667617,1667633,1667637,1667747,1667767,1667873,1668028,1668137,1668634,1669432,1669801,1669840,1669895-1669896,1670398,1670435,1670592,1670605-1670607,1670609,1670632,1670720,1670725,1670727,1670731,1671114,1672273,1672285,1673759,1674220,1674295,1675469,1675488,1675595,1675831,1676232,1676367-1676369,1676382,1676394,1676483,1676556,1676635,1678178,1679536,1679988,1680256,1681124,1681182,1681703,1681730,1681840,1681864,1681869,1682010,1682034,1682047,1682052-1682053,1682062,1682064,1682070,1682312,1682325,1682331,1682386,1684367,1684385,1685759,1685774,1685827,1685892,1687341,1688904,1689358,1689657,1689921,1692850,1693093,1693108,1693324,1694060,1694115,1694291,1694427,1694431,1694503,1694549,1694789,1694873,1694881,1695356,1695372,1695823-1695825,1696200,1696281,1696379,1696468,1700608,1700871,1700897,1700978,1701094,1701124,1701608,1701668,1701676,1701766,1701944,1702248,1702252,1702314,1702390,1702723,1702725,1702728,1702730,1702733,1702735,1702737,1702739,1702
 
742,1702744,1702748,1702751,1702754,1702758,1702760,1702763,1702766,1708779,1708782,1708806,1709314,1709670,1710347,1710442,1710448,1710490,1710574,1710578,1712226,1712229,1712235,1712255,1712618,1712649,1712655,1712860,1712899,1712903,1712906,1712913,1712926,1712975,1713185,1713262,1713287,1713613,1713621,1713872,1713976,1713994,1713998,1714004,1714013,1714059,1714538,1714580,1715189,1715207,1715544,1715549,1715637,1715639-1715645,1715667,1715683,1715866,1715978,1715981,1716216-1716217,1716355,1716414,1716421,1717208-1717209,1717257,1717283,1717288,1717291,1717421,1717517,1717529,1718797,1718840-1718843,1719348,1719357-1719358,1719400,1719491,1719737,1720235,1720396,1720442,1720446,1720450,1720463,1720658-1720660,1720756,1720816,1721813,1721818,1721831,1721861,1721867,1721882,1722523,1722527,1722800,1722926,1722941,1722997,1723130,1723440,1723488,1723890,1724434,1724674,1724792,1724803,1724902,1725128,1725131,1725154,1725167,1725911,1725921,1725929,1725963-1725965,1725970,1725974,1
 
726171-1726173,1726175,1726179-1726182,1726190-1726191,1726195-1726200,1726203,1726226,1726576,1726630,1726992,1727029,1727037,1727671,1727676,1727900,1728028,1728092,1728439,1728449,1729186,1729362,1731009,1731303,1731867,1731872,1731874,1731876,1731885,1731947,1731955,1731959,1731977,1731984,1732360,1732490,1732672,1732902,1733166,1733603,1733619,1733735,1733752,1733764,1733915,1733941,1733964,1734115,1734133,1734261,1734421,1734531,1736286,1737967,1738173,1738182,1738992,1739039,1739089-1739091,1739294,1739777,1739821,1739981,1740513,1740726,1741019,1741162,1741217,1743647,1743681,1744152,1744272,1746732,1746750,1752739,1754615,1755886,1756018,1759565,1761686,1762173,1762206,1766280,1767507-1767508,1767653,1767656,1769267,1772949,1773521,1773527,1774104,1777015,1777213
-/tomcat/tc8.5.x/trunk:1735579,1736839,1737199,1737966,1738042,1738044,1738162,1738165,1738178,1739157,1739173,1739177,1739476,1740132,1740521,1740536,1740804,174081

[GitHub] tomcat pull request #37: Spelling fixes

2017-01-09 Thread jsoref
GitHub user jsoref opened a pull request:

https://github.com/apache/tomcat/pull/37

Spelling fixes

* abandoned
* adapter
* application
* approach
* assign
* assuming
* async
* asynchronous
* attempt
* attribute
* authoritative
* authorization
* available
* background
* beginning
* being
* belonging
* blocking
* boolean
* broadcast
* cacheable
* calendar
* candidate
* character
* classes
* classpath
* collected
* complex
* concurrent
* configured
* connection
* constant
* constraint
* containing
* context
* couldn't
* coyote
* created
* customization
* customize
* deflate
* dependency
* deployment
* describing
* description
* descriptor
* destroy
* digest
* directory
* documentation
* doesn't
* empty
* encodeable
* even though
* exception
* experiences
* explicitly
* expressions
* extension
* externally
* factory
* fallback
* following
* formats
* formatter
* forwarded
* fujitsu
* function
* functionality
* fundamental
* further
* handler
* handshake
* iface
* ignore
* implementation
* implicit
* important
* independently
* individually
* initialization
* initialized
* initializer
* initially
* initiated
* instance
* instantiation
* interceptor
* interest
* interfaces
* interpret
* interrupt
* interrupted
* invocation
* language
* length
* library
* lifecycle
* localpart
* manifest
* maucherat
* message
* messages
* metadata
* multicast
* multiple
* namespace
* normalized
* notification
* precondition
* preemptive
* prefix
* presence
* principal's
* privileges
* programmatically
* pseudo
* public
* received
* references
* registration
* registry
* reinstantiating
* related
* remember
* renegotiation
* representation
* requested
* required
* resetting
* resource
* response
* result
* rewrite
* running
* semantics
* serializations
* servlet
* session
* shouldn't
* significant
* simultaneously
* singing
* skipping
* special
* specified
* specifying
* standard
* stopped
* substitute
* support
* supported
* syntax
* system
* tag
* threshold
* tomcat
* trailer
* trivial
* undeclared
* undeploy
* underlying
* unique
* unknown
* usually
* variable
* version
* weren't
* writable
* write
* writing

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jsoref/tomcat spelling

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/tomcat/pull/37.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #37


commit 8e3516bb5944359a3bde5e0e7b7bd6d2999c7846
Author: Josh Soref 
Date:   2017-01-10T05:45:37Z

Spelling fixes

* abandoned
* adapter
* application
* approach
* assign
* assuming
* async
* asynchronous
* attempt
* attribute
* authoritative
* authorization
* available
* background
* beginning
* being
* belonging
* blocking
* boolean
* broadcast
* cacheable
* calendar
* candidate
* character
* classes
* classpath
* collected
* complex
* concurrent
* configured
* connection
* constant
* constraint
* containing
* context
* couldn't
* coyote
* created
* customization
* customize
* deflate
* dependency
* deployment
* describing
* description
* descriptor
* destroy
* digest
* directory
* documentation
* doesn't
* empty
* encodeable
* even though
* exception
* experiences
* explicitly
* expressions
* extension
* externally
* factory
* fallback
* following
* formats
* formatter
* forwarded
* fujitsu
* function
* functionality
* fundamental
* further
* handler
* handshake
* iface
* ignore
* implementation
* implicit
* important
* independently
* individually
* initialization
* initialized
* initializer
* initially
* initiated
* instance
* instantiation
* interceptor
* interest
* interfaces
* interpret
* interrupt
* interrupted
* invocation
* language
* length
* library
* lifecycle
* localpart
* manifest
* maucherat
* message
* messages
* me

svn commit: r1778090 - in /tomcat/trunk: modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ webapps/docs/

2017-01-09 Thread kfujino
Author: kfujino
Date: Tue Jan 10 07:22:43 2017
New Revision: 1778090

URL: http://svn.apache.org/viewvc?rev=1778090&view=rev
Log:
Enable reset the statistics without restarting the pool.

Modified:

tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java

tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java

tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java

tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java

tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/mbeans-descriptors.xml
tomcat/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java?rev=1778090&r1=1778089&r2=1778090&view=diff
==
--- 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 (original)
+++ 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 Tue Jan 10 07:22:43 2017
@@ -1251,6 +1251,19 @@ public class ConnectionPool {
 }
 
 /**
+ * reset the statistics of this pool.
+ */
+public void resetStats() {
+borrowedCount.set(0);
+returnedCount.set(0);
+createdCount.set(0);
+releasedCount.set(0);
+reconnectedCount.set(0);
+removeAbandonedCount.set(0);
+releasedIdleCount.set(0);
+}
+
+/**
  * Tread safe wrapper around a future for the regular queue
  * This one retrieves the pooled connection object
  * and performs the initialization according to

Modified: 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java?rev=1778090&r1=1778089&r2=1778090&view=diff
==
--- 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
 (original)
+++ 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
 Tue Jan 10 07:22:43 2017
@@ -837,6 +837,17 @@ public class DataSourceProxy implements
 }
 }
 
+/**
+ * reset the statistics of this pool.
+ */
+public void resetStats() {
+try {
+createPool().resetStats();
+} catch (SQLException x) {
+throw new RuntimeException(x);
+}
+}
+
 //=
 //  PROPERTIES / CONFIGURATION
 //=

Modified: 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java?rev=1778090&r1=1778089&r2=1778090&view=diff
==
--- 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 (original)
+++ 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 Tue Jan 10 07:22:43 2017
@@ -218,6 +218,12 @@ public class ConnectionPool extends Noti
 public void testIdle() {
 pool.testAllIdle();
 }
+
+@Override
+public void resetStats() {
+pool.resetStats();
+}
+
 //=
 //   POOL PROPERTIES
 //=

Modified: 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java?rev=1778090&r1=1778089&r2=1778090&view=diff
==
--- 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
 (original)
+++ 
tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
 Tue Jan 10 07:22:43 2017
@@ -74,6 +74,11 @@ public interface ConnectionPoolMBean ext
  */
 public void purgeOnReturn();
 
+/**
+ * reset the statistics of this pool.
+ */
+public void resetStats();
+
 //=
 //   POOL NOTIFICATIONS
 //

svn commit: r1778091 - in /tomcat/tc8.5.x/trunk: modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ webapps/docs/

2017-01-09 Thread kfujino
Author: kfujino
Date: Tue Jan 10 07:26:12 2017
New Revision: 1778091

URL: http://svn.apache.org/viewvc?rev=1778091&view=rev
Log:
Enable reset the statistics without restarting the pool.

Modified:

tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java

tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java

tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java

tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java

tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/mbeans-descriptors.xml
tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java?rev=1778091&r1=1778090&r2=1778091&view=diff
==
--- 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 (original)
+++ 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 Tue Jan 10 07:26:12 2017
@@ -1251,6 +1251,19 @@ public class ConnectionPool {
 }
 
 /**
+ * reset the statistics of this pool.
+ */
+public void resetStats() {
+borrowedCount.set(0);
+returnedCount.set(0);
+createdCount.set(0);
+releasedCount.set(0);
+reconnectedCount.set(0);
+removeAbandonedCount.set(0);
+releasedIdleCount.set(0);
+}
+
+/**
  * Tread safe wrapper around a future for the regular queue
  * This one retrieves the pooled connection object
  * and performs the initialization according to

Modified: 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java?rev=1778091&r1=1778090&r2=1778091&view=diff
==
--- 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
 (original)
+++ 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
 Tue Jan 10 07:26:12 2017
@@ -824,6 +824,7 @@ public class DataSourceProxy implements
 throw new RuntimeException(x);
 }
 }
+
 /**
  * The total number of connections reconnected by this pool.
  * @return the reconnected connection count
@@ -835,6 +836,18 @@ public class DataSourceProxy implements
 throw new RuntimeException(x);
 }
 }
+
+/**
+ * reset the statistics of this pool.
+ */
+public void resetStats() {
+try {
+createPool().resetStats();
+} catch (SQLException x) {
+throw new RuntimeException(x);
+}
+}
+
 //=
 //  PROPERTIES / CONFIGURATION
 //=

Modified: 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java?rev=1778091&r1=1778090&r2=1778091&view=diff
==
--- 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 (original)
+++ 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 Tue Jan 10 07:26:12 2017
@@ -218,6 +218,12 @@ public class ConnectionPool extends Noti
 public void testIdle() {
 pool.testAllIdle();
 }
+
+@Override
+public void resetStats() {
+pool.resetStats();
+}
+
 //=
 //   POOL PROPERTIES
 //=

Modified: 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java?rev=1778091&r1=1778090&r2=1778091&view=diff
==
--- 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
 (original)
+++ 
tomcat/tc8.5.x/trunk/modules/jdbc-pool/s

svn commit: r1778092 - in /tomcat/tc8.0.x/trunk: modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ webapps/docs/

2017-01-09 Thread kfujino
Author: kfujino
Date: Tue Jan 10 07:28:57 2017
New Revision: 1778092

URL: http://svn.apache.org/viewvc?rev=1778092&view=rev
Log:
Enable reset the statistics without restarting the pool.

Modified:

tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java

tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java

tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java

tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java

tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/mbeans-descriptors.xml
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java?rev=1778092&r1=1778091&r2=1778092&view=diff
==
--- 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 Tue Jan 10 07:28:57 2017
@@ -1240,6 +1240,19 @@ public class ConnectionPool {
 }
 
 /**
+ * reset the statistics of this pool.
+ */
+public void resetStats() {
+borrowedCount.set(0);
+returnedCount.set(0);
+createdCount.set(0);
+releasedCount.set(0);
+reconnectedCount.set(0);
+removeAbandonedCount.set(0);
+releasedIdleCount.set(0);
+}
+
+/**
  * Tread safe wrapper around a future for the regular queue
  * This one retrieves the pooled connection object
  * and performs the initialization according to

Modified: 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java?rev=1778092&r1=1778091&r2=1778092&view=diff
==
--- 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
 Tue Jan 10 07:28:57 2017
@@ -798,6 +798,7 @@ public class DataSourceProxy implements
 throw new RuntimeException(x);
 }
 }
+
 /**
  * The total number of connections reconnected by this pool.
  * @return the reconnected connection count
@@ -809,6 +810,18 @@ public class DataSourceProxy implements
 throw new RuntimeException(x);
 }
 }
+
+/**
+ * reset the statistics of this pool.
+ */
+public void resetStats() {
+try {
+createPool().resetStats();
+} catch (SQLException x) {
+throw new RuntimeException(x);
+}
+}
+
 //=
 //  PROPERTIES / CONFIGURATION
 //=

Modified: 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java?rev=1778092&r1=1778091&r2=1778092&view=diff
==
--- 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 Tue Jan 10 07:28:57 2017
@@ -218,6 +218,12 @@ public class ConnectionPool extends Noti
 public void testIdle() {
 pool.testAllIdle();
 }
+
+@Override
+public void resetStats() {
+pool.resetStats();
+}
+
 //=
 //   POOL PROPERTIES
 //=

Modified: 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java?rev=1778092&r1=1778091&r2=1778092&view=diff
==
--- 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/modules/jdbc-pool/s

svn commit: r1778093 - in /tomcat/tc7.0.x/trunk: modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ webapps/docs/

2017-01-09 Thread kfujino
Author: kfujino
Date: Tue Jan 10 07:31:31 2017
New Revision: 1778093

URL: http://svn.apache.org/viewvc?rev=1778093&view=rev
Log:
Enable reset the statistics without restarting the pool.

Modified:

tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java

tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java

tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java

tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java

tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/mbeans-descriptors.xml
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java?rev=1778093&r1=1778092&r2=1778093&view=diff
==
--- 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 Tue Jan 10 07:31:31 2017
@@ -1243,6 +1243,19 @@ public class ConnectionPool {
 }
 
 /**
+ * reset the statistics of this pool.
+ */
+public void resetStats() {
+borrowedCount.set(0);
+returnedCount.set(0);
+createdCount.set(0);
+releasedCount.set(0);
+reconnectedCount.set(0);
+removeAbandonedCount.set(0);
+releasedIdleCount.set(0);
+}
+
+/**
  * Tread safe wrapper around a future for the regular queue
  * This one retrieves the pooled connection object
  * and performs the initialization according to

Modified: 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java?rev=1778093&r1=1778092&r2=1778093&view=diff
==
--- 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/DataSourceProxy.java
 Tue Jan 10 07:31:31 2017
@@ -784,6 +784,7 @@ public class DataSourceProxy implements
 throw new RuntimeException(x);
 }
 }
+
 /**
  * The total number of connections reconnected by this pool.
  * @return the reconnected connection count
@@ -795,6 +796,18 @@ public class DataSourceProxy implements
 throw new RuntimeException(x);
 }
 }
+
+/**
+ * reset the statistics of this pool.
+ */
+public void resetStats() {
+try {
+createPool().resetStats();
+} catch (SQLException x) {
+throw new RuntimeException(x);
+}
+}
+
 //=
 //  PROPERTIES / CONFIGURATION
 //=

Modified: 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java?rev=1778093&r1=1778092&r2=1778093&view=diff
==
--- 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 Tue Jan 10 07:31:31 2017
@@ -219,6 +219,12 @@ public class ConnectionPool extends Noti
 public void testIdle() {
 pool.testAllIdle();
 }
+
+@Override
+public void resetStats() {
+pool.resetStats();
+}
+
 //=
 //   POOL PROPERTIES
 //=

Modified: 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java?rev=1778093&r1=1778092&r2=1778093&view=diff
==
--- 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPoolMBean.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/s