apache-tomcat-7.0.56/webapps/docs/architecture/requestProcess/requestProcess.pdf
it is about the request process UML diagram, I have been reading tomcat 7.0.56 source code for weeks, and along with source code reading along with debug which helps so much, and one day after I debgued the whole end-to-end request flow, I found that the document seems is derived from Tomcat 5(please check the footer of the pdf).
and there are few steps on the diagram which are different from my finding/debuging.
Please kindly check attachments, thanks!

(In reply to Mark Thomas from comment #3)
> Yes, those diagrams are out of date.
>
> The problem is bigger than you have identified. Page 2 is completely wrong.
> The call sequence is very different.
Hi Mark,
community would fix that in the future? or what can I help ?

The calls prior to CoyoteAdapter don't look right. Specifically, there is no Http11Processor and no SocketProcessor. After that things look OK. I do wonder if it would be more useful with an Authenticator + Realm configured. Maybe that should/could be in a separate diagram.

(In reply to Mark Thomas from comment #8)
> The calls prior to CoyoteAdapter don't look right. Specifically, there is no
> Http11Processor and no SocketProcessor. After that things look OK. I do
> wonder if it would be more useful with an Authenticator + Realm configured.
> Maybe that should/could be in a separate diagram.
Mark, I was thinking about Http11Protocol represents both SockeProcessor(before Http11Protocol) and Http11Processor(after Http11Protocol) which is not right, the call hierarchy is very clear(at in the source code) and it should be reflected in the diagram accordingly.
thanks for your comments, would supplement missed part, and as your said authenticator + Realm diagram would be very useful, would supplement accordingly too.
Regards

Broadly it looks OK but the detail around the Pipelines isn't right. Calls to getFirst() are missing and some method names shown don't exist. Also methods are being called on the wrong objects (e.g. getHost() on the StandardHost).
Similar comments apply to the Authentication diagram. For example getContext() is shown as being called on StandardContext.
I'd consider ignoring the various getHost(), getPipeline() calls and just show the calls that do the real work.

(In reply to Mark Thomas from comment #13)
> Broadly it looks OK but the detail around the Pipelines isn't right. Calls
> to getFirst() are missing and some method names shown don't exist. Also
> methods are being called on the wrong objects (e.g. getHost() on the
> StandardHost).
>
> Similar comments apply to the Authentication diagram. For example
> getContext() is shown as being called on StandardContext.
>
> I'd consider ignoring the various getHost(), getPipeline() calls and just
> show the calls that do the real work.
Mark, thanks for your valuable comment as a reader.
That diagram wasn't that strict, since in my mind, I wanted to express the process in a verbose&abstract way which is confusing people.
I have removed those unnecessary methods(getHost, getContext...) from previous diagrams, please check the latest ones and give comments, would be delighted to help.

The request processing diagram looks really good. There is one issue that needs fixing before we can use it. At the end of the chain Servlet.service() is called from ApplicationFilterChain.doFilter(), not ApplicationFilterChain.doRelease(). Get that fixed and I think it will be ready to commit.

The authentication diagram also looks good. I do have a few more comments on this.
registerAuthSuccess() is specific to the LockOutRealm. It would be worth showing this Realm and the 'real' Realm (e.g. UserDataBaseRealm) on the diagram.
For BASIC auth I'd add the call to obtain the headers from the request since that is where the user name and password are called from.

Hello Mark,
Good to hear from you again, for the request process diagram, that was a typo, andfixed now.
for the authenticate diagram, revised with you nice suggestions, please have a look and review, thanks.

(In reply to Mark Thomas from comment #26)
> These have been added to the documentation for trunk (9.0.x), 8.0.x (for
> 8.0.24 onwards) and 7.0.x (for 7.0.63 onwards).
>
> Many thanks for all your hard work.
Yeah, I am so honored&thrilled what I have done.
Of course, many thanks for your great help, (In reply to Mark Thomas from comment #26)
> These have been added to the documentation for trunk (9.0.x), 8.0.x (for
> 8.0.24 onwards) and 7.0.x (for 7.0.63 onwards).
>
> Many thanks for all your hard work.
Yeah!!! I am so thrilled&honored about what I have done for community.
of course, many thanks for your great help(valuable reviews and comments), it can't be done without your help.

This is ASF Bugzilla: the Apache Software Foundation bug system. In case
of problems with the functioning of ASF Bugzilla, please contact
bugzilla-admin@apache.org.
Please Note: this e-mail address is only for reporting problems
with ASF Bugzilla. Mail about any other subject will be silently
ignored.