Mark Struberg created MYFACES-3652:
--------------------------------------

             Summary: Define default view key algorithm
                 Key: MYFACES-3652
                 URL: https://issues.apache.org/jira/browse/MYFACES-3652
             Project: MyFaces Core
          Issue Type: Sub-task
          Components: JSR-344
    Affects Versions: 2.1.9, 2.2.0
            Reporter: Mark Struberg
            Assignee: Mark Struberg


Currently we have a few different viewkey generator implementations. Those got 
added only in 2.1.9. Before that the only had a TicketCounter in each Session. 

The original implementation also had no viewId in the key.
If you think about it, then it makes no sense at all to add the viewId. Despite 
it's an int hashCode we have 2 problems which completely trashes the purpose: 
a.) hashCode is not guaranteed to be unique
b.) the hashCode is always the same for the same view.

Think about an application with only one xhtml page. In that case the viewId 
would add no additional info.
With 4 pages you would only reduce the collision rate to over 25%. Since the 
application will most times mainly hit by a few entry points like index.html 
you gain barely anything from adding this information.

IF we have had problems with any collisions, then we shall add an XorShift 
random generator instead of the viewId. Leo, I didn't an issue report for such 
a problem. Do you have any tip for me where I can find that?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to