cvs commit: xml-cocoon2/src/java/org/apache/cocoon/transformation FragmentExtractorTransformer.java

[sylvain]

sylvain     02/01/23 12:58:11

  Modified:    .        todo.xml
               src/java/org/apache/cocoon/generation
                        FragmentExtractorGenerator.java
               src/java/org/apache/cocoon/transformation
                        FragmentExtractorTransformer.java
  Log:
  Add warning about clustered engines, and plan a redesign.
  
  Revision  Changes    Path
  1.22      +8 -1      xml-cocoon2/todo.xml
  
  Index: todo.xml
  ===================================================================
  RCS file: /home/cvs/xml-cocoon2/todo.xml,v
  retrieving revision 1.21
  retrieving revision 1.22
  diff -u -r1.21 -r1.22
  --- todo.xml  23 Jan 2002 10:31:27 -0000      1.21
  +++ todo.xml  23 Jan 2002 20:58:11 -0000      1.22
  @@ -4,7 +4,7 @@
   
   <!--
     History of Cocoon changes
  -  $Id: todo.xml,v 1.21 2002/01/23 10:31:27 cziegeler Exp $
  +  $Id: todo.xml,v 1.22 2002/01/23 20:58:11 sylvain Exp $
   -->
   
   
  @@ -21,6 +21,7 @@
     <person name="Giacomo Pati" email="[EMAIL PROTECTED]" id="GP"/>
     <person name="Ricardo Rocha" email="[EMAIL PROTECTED]" id="RR"/>
     <person name="Davanum Srinivas" email="[EMAIL PROTECTED]" id="DM"/>
  +  <person name="Sylvain Wallez" email="[EMAIL PROTECTED]" id="SW"/>
     <person name="Carsten Ziegeler" email="[EMAIL PROTECTED]" id="CZ"/>
     <person name="Gerhard Fr&#246;hlich" email="[EMAIL PROTECTED]" id="GF"/>
     <person name="John Morrison" email="[EMAIL PROTECTED]" id="JM"/>
  @@ -95,6 +96,12 @@
      definitions from cocoon.xconf into sitemap.xmap, &lt;map:components&gt;
      section; and (2) allow usage of different pipeline implementations in
      &lt;map:pipelines&gt; section of the sitemap.
  +  </action>
  +
  +  <action context="code">
  +   Redesign FragmentExtractorGenerator/Transformer so that it works on a clustered
  +   server : store fragments in the session rather than in a local store. <br/>
  +   Additionnaly, store the fragments using the XMLCompiler instead of building a 
DOM.
     </action>
    </actions>
   
  
  
  
  1.3       +5 -1      
xml-cocoon2/src/java/org/apache/cocoon/generation/FragmentExtractorGenerator.java
  
  Index: FragmentExtractorGenerator.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/generation/FragmentExtractorGenerator.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- FragmentExtractorGenerator.java   8 Jan 2002 10:47:45 -0000       1.2
  +++ FragmentExtractorGenerator.java   23 Jan 2002 20:58:11 -0000      1.3
  @@ -35,8 +35,12 @@
    *
    * This is by no means complete yet, but it should prove useful, particularly
    * for offline generation.
  + * <p>
  + * <b><em>Warning</em> : since fragments are stored locally in the class, this 
generator
  + * and the associated transformer are very likely to fail on a clustered server.</b>
  + *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Paul Russell</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/08 10:47:45 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/23 20:58:11 $
    */
   public class FragmentExtractorGenerator extends AbstractGenerator
           implements Recyclable, Cacheable {
  
  
  
  1.3       +5 -1      
xml-cocoon2/src/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java
  
  Index: FragmentExtractorTransformer.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- FragmentExtractorTransformer.java 8 Jan 2002 10:47:46 -0000       1.2
  +++ FragmentExtractorTransformer.java 23 Jan 2002 20:58:11 -0000      1.3
  @@ -41,8 +41,12 @@
    * mainly SVG extraction that's likely to go on, and afterall,
    * <a href="http://c2.com/cgi/wiki?YouArentGonnaNeedIt";>you aren't gonna need 
it</a>,
    * so I've just used very simple extraction based on a URI and local name.
  + * <p>
  + * <b><em>Warning</em> : since fragments are stored locally in the class, this 
transformer
  + * and the associated generator are very likely to fail on a clustered server.</b>
  + *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Paul Russell</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/08 10:47:46 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/23 20:58:11 $
    */
   public class FragmentExtractorTransformer extends AbstractTransformer
       implements Composable, Disposable, Cacheable, Poolable {
  
  
  

----------------------------------------------------------------------
In case of troubles, e-mail:     [EMAIL PROTECTED]
To unsubscribe, e-mail:          [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]