myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeanne Waldman <jeanne.wald...@oracle.com>
Subject Re: [Trinidad 2.0] introducing ProjectStage API
Date Thu, 10 Dec 2009 18:08:35 GMT
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
I agree with your list Matthias.<br>
Jeanne<br>
<br>
Matthias Wessendorf wrote, On 12/8/2009 8:56 AM PT:
<blockquote
 cite="mid:71235db40912080856v21cdc1e1jbe47ce49b874c540@mail.gmail.com"
 type="cite">
  <pre wrap=""><a class="moz-txt-link-freetext" href="http://wiki.apache.org/myfaces/Trinidad_goes_ProjectStage">http://wiki.apache.org/myfaces/Trinidad_goes_ProjectStage</a>

On Tue, Dec 8, 2009 at 5:54 PM, Matthias Wessendorf <a class="moz-txt-link-rfc2396E" href="mailto:matzew@apache.org">&lt;matzew@apache.org&gt;</a>
wrote:
  </pre>
  <blockquote type="cite">
    <pre wrap="">ok,

I updated the wiki:

I will continue with these values, as there was no concern against them.

regarding the hidden, unspecified
"javax.faces.FACELETS_REFRESH_PERIOD" parameter, I will follow up in
a different thread.

-Matthias

On Wed, Dec 2, 2009 at 9:06 PM, Matthias Wessendorf <a class="moz-txt-link-rfc2396E" href="mailto:matzew@apache.org">&lt;matzew@apache.org&gt;</a>
wrote:
    </pre>
    <blockquote type="cite">
      <pre wrap="">On Wed, Dec 2, 2009 at 8:52 PM, Pavitra Subramaniam
<a class="moz-txt-link-rfc2396E" href="mailto:pavitra.subramaniam@oracle.com">&lt;pavitra.subramaniam@oracle.com&gt;</a>
wrote:
      </pre>
      <blockquote type="cite">
        <pre wrap="">"Matthias Wessendorf wrote:" On 12/2/2009 10:43 AM PT:

Ok,

I added some value suggestions to those guys that we identified to fit
into the ProjectStage API:


-org.apache.myfaces.trinidad.DEBUG_JAVASCRIPT
 * Production =&gt; FALSE
 * other stages =&gt; TRUE


+1. although I think if a user were to change the recommended defaults, like
set ProjectStage=Production and override the DEBUG_JS to be true, then we at
least need to warn the user of this as appropriate.
        </pre>
      </blockquote>
      <pre wrap="">Yeah I was planing do WARNING for everything that is overridden

      </pre>
      <blockquote type="cite">
        <pre wrap="">-org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION
 * Production =&gt; FALSE
 * other stages =&gt; TRUE
-org.apache.myfaces.trinidad.CHECK_FILE_MODIFICATION
 * Production =&gt; FALSE
 * other stages =&gt; TRUE
-org.apache.myfaces.trinidad.resource.DEBUG
 * Production =&gt; FALSE
 * other stages =&gt; TRUE
-org.apache.myfaces.trinidadinternal.ENABLE_PPR_OPTIMIZATION (see above)
 * Production =&gt; FALSE (current default)
 * other stages =&gt; TRUE
-&lt;debug-output&gt; (from trinidad-config.xml file)
 * Production =&gt; FALSE
 * other stages =&gt; TRUE


+1. I also was wondering if
"org.apache.myfaces.trinidad.CHECK_STATE_SERIALIZATION" should be set to
NONE in Production if this is really used for a testing Trinidad in
pseudo-High Availability (HA) mode. It looks like we could set this
different values based on the the ProjectStage

Production
org.apache.myfaces.trinidad.CHECK_STATE_SERIALIZATION=none

Development, Unit Test
org.apache.myfaces.trinidad.CHECK_STATE_SERIALIZATION=session,tree

System Test
org.apache.myfaces.trinidad.CHECK_STATE_SERIALIZATION=all
        </pre>
      </blockquote>
      <pre wrap="">great. thanks for the feedback. Makes sense to me. I think that this
parameter actually *translates* to the ProjectStage.
Your suggestions make sense to me.

-Matthias


      </pre>
      <blockquote type="cite">
        <pre wrap="">-Pavitra

Please let me know what you think...


I updated the wiki page, to reflect the suggestions:
<a class="moz-txt-link-freetext" href="http://wiki.apache.org/myfaces/Trinidad_goes_ProjectStage">http://wiki.apache.org/myfaces/Trinidad_goes_ProjectStage</a>

I also created this JIRA ticket, to keep track of the item:
<a class="moz-txt-link-freetext" href="https://issues.apache.org/jira/browse/TRINIDAD-1652">https://issues.apache.org/jira/browse/TRINIDAD-1652</a>

-Matthias

On Wed, Dec 2, 2009 at 3:18 PM, Matthias Wessendorf <a class="moz-txt-link-rfc2396E" href="mailto:matzew@apache.org">&lt;matzew@apache.org&gt;</a>
wrote:


yep, that's what I was saying

On Wed, Dec 2, 2009 at 3:04 PM, Jakob Korherr <a class="moz-txt-link-rfc2396E" href="mailto:jakob.korherr@gmail.com">&lt;jakob.korherr@gmail.com&gt;</a>
wrote:


Hi,

I think the best way would be that the init parameters have stage related
standard values (actually that what Matthias suggested), for example:

If we are in ProjectStage.Production the standard value of
org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION is false, otherwise
it is true.

But this only sets the DEFAULT value. The user still can configure each
parameter independently by putting it in web.xml.

Regards

Jakob Korherr

2009/12/2 Martin Koci <a class="moz-txt-link-rfc2396E" href="mailto:martin.koci@aura.cz">&lt;martin.koci@aura.cz&gt;</a>


Hi,

does trinidad reload both JSP and facelets views if
org.apache.myfaces.trinidad.CHECK_FILE_MODIFICATION=true?

For facelets based views there is a new JSF 2.0 param
javax.faces.FACELETS_REFRESH_PERIOD (-1 = never reload, 0=always, 1,2,
3,.. - check interval) but it's relation to project stage is not clear -
see <a class="moz-txt-link-freetext" href="https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1434">https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1434</a>

Martin Kočí

Matthias Wessendorf píše v St 02. 12. 2009 v 09:25 +0100:


Hi,

here is a (complete?) list of all Trinidad configuration parameters:

* System Properties ( =&gt; System.getProperty(....)):
org.apache.myfaces.trinidad.CHECK_STATE_SERIALIZATION
documentation is here:
<a class="moz-txt-link-freetext" href="http://issues.apache.org/jira/browse/TRINIDAD-1245">http://issues.apache.org/jira/browse/TRINIDAD-1245</a>

<a class="moz-txt-link-freetext" href="http://myfaces.apache.org/trinidad/trinidad-api/apidocs/org/apache/myfaces/trinidad/bean/util/StateUtils.html">http://myfaces.apache.org/trinidad/trinidad-api/apidocs/org/apache/myfaces/trinidad/bean/util/StateUtils.html</a>

==&gt; does not really make sense to port to Stage.....

* The "trinidad-config.xml" file:
See here for infos about it:
<a class="moz-txt-link-freetext" href="http://myfaces.apache.org/trinidad/devguide/configuration.html">http://myfaces.apache.org/trinidad/devguide/configuration.html</a>

I think that ONLY this element could make sense:
-&lt;debug-output&gt;
(e.g. as for PRODUCTION you want FALSE here... )

* Settings in the WEB.XML (aka Context Parameters):
Some info is here as well:
<a class="moz-txt-link-freetext" href="http://myfaces.apache.org/trinidad/devguide/configuration.html">http://myfaces.apache.org/trinidad/devguide/configuration.html</a>

The web.xml has different types of settings:
some fit to ProjectStage, some not......

Here is a grouped list, where I think they don't make sense with
project_stage at all:
-org.apache.myfaces.trinidad.CACHE_VIEW_ROOT
-org.apache.myfaces.trinidad.CLIENT_STATE_METHOD
-org.apache.myfaces.trinidad.CLIENT_STATE_MAX_TOKENS
-org.apache.myfaces.trinidad.CHANGE_PERSISTENCE
-org.apache.myfaces.trinidad.USE_APPLICATION_VIEW_CACHE
-org.apache.myfaces.trinidad.ENABLE_LIGHTWEIGHT_DIALOGS
-org.apache.myfaces.trinidad.ALTERNATE_VIEW_HANDLER
-org.apache.myfaces.trinidad.FACELETS_VIEW_MAPPINGS
-org.apache.myfaces.trinidad.UPLOAD_MAX_MEMORY
-org.apache.myfaces.trinidad.UPLOAD_MAX_DISK_SPACE
-org.apache.myfaces.trinidad.UPLOAD_TEMP_DIR

These parameters do make sense:
-org.apache.myfaces.trinidad.DEBUG_JAVASCRIPT
-org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION
-org.apache.myfaces.trinidad.CHECK_FILE_MODIFICATION
-org.apache.myfaces.trinidad.resource.DEBUG
(note, we need a bug here, as this is NOT documented in the "official
documentation" :-) )

This parameter _may_ make sense (*caution:* it is an internal and
temporary param):
-org.apache.myfaces.trinidadinternal.ENABLE_PPR_OPTIMIZATION
See <a class="moz-txt-link-freetext" href="http://issues.apache.org/jira/browse/TRINIDAD-1368">http://issues.apache.org/jira/browse/TRINIDAD-1368</a>

So basically, only these make sense (for me):
-org.apache.myfaces.trinidad.DEBUG_JAVASCRIPT
-org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION
-org.apache.myfaces.trinidad.CHECK_FILE_MODIFICATION
-org.apache.myfaces.trinidad.resource.DEBUG
-org.apache.myfaces.trinidadinternal.ENABLE_PPR_OPTIMIZATION (*see
above)
-&lt;debug-output&gt;

Let me know if you agree with this list and I will follow up on what
values they should have,
in what STAGE....

A note: Perhaps we want to be able to still override a _SINGLE_
setting, regardless for the
current STAGE. Means: If the application runs in
"ProjectStage.Production", we do CSS compression.
BUT... if the web.xml says
"org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION =&gt; TRUE",
we should honor that... This would give users a more fine grained
control over these configuration settings.

What do you think ?

Greetings,
Matthias








--
Matthias Wessendorf

blog: <a class="moz-txt-link-freetext" href="http://matthiaswessendorf.wordpress.com/">http://matthiaswessendorf.wordpress.com/</a>
sessions: <a class="moz-txt-link-freetext" href="http://www.slideshare.net/mwessendorf">http://www.slideshare.net/mwessendorf</a>
twitter: <a class="moz-txt-link-freetext" href="http://twitter.com/mwessendorf">http://twitter.com/mwessendorf</a>




        </pre>
      </blockquote>
      <pre wrap="">

--
Matthias Wessendorf

blog: <a class="moz-txt-link-freetext" href="http://matthiaswessendorf.wordpress.com/">http://matthiaswessendorf.wordpress.com/</a>
sessions: <a class="moz-txt-link-freetext" href="http://www.slideshare.net/mwessendorf">http://www.slideshare.net/mwessendorf</a>
twitter: <a class="moz-txt-link-freetext" href="http://twitter.com/mwessendorf">http://twitter.com/mwessendorf</a>

      </pre>
    </blockquote>
    <pre wrap="">

--
Matthias Wessendorf

blog: <a class="moz-txt-link-freetext" href="http://matthiaswessendorf.wordpress.com/">http://matthiaswessendorf.wordpress.com/</a>
sessions: <a class="moz-txt-link-freetext" href="http://www.slideshare.net/mwessendorf">http://www.slideshare.net/mwessendorf</a>
twitter: <a class="moz-txt-link-freetext" href="http://twitter.com/mwessendorf">http://twitter.com/mwessendorf</a>

    </pre>
  </blockquote>
  <pre wrap=""><!---->


  </pre>
</blockquote>
</body>
</html>

Mime
View raw message