jmeter-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pmoua...@apache.org
Subject svn commit: r1530078 - in /jmeter/trunk: bin/jmeter.properties src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java
Date Mon, 07 Oct 2013 21:40:50 GMT
Author: pmouawad
Date: Mon Oct  7 21:40:50 2013
New Revision: 1530078

URL: http://svn.apache.org/r1530078
Log:
Bug 55632 - Have a new implementation of htmlParser for embedded resources parsing with better
performances
Switch default to Lagarto Parser implementation
Bugzilla Id: 55632

Modified:
    jmeter/trunk/bin/jmeter.properties
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java

Modified: jmeter/trunk/bin/jmeter.properties
URL: http://svn.apache.org/viewvc/jmeter/trunk/bin/jmeter.properties?rev=1530078&r1=1530077&r2=1530078&view=diff
==============================================================================
--- jmeter/trunk/bin/jmeter.properties (original)
+++ jmeter/trunk/bin/jmeter.properties Mon Oct  7 21:40:50 2013
@@ -575,17 +575,20 @@ HTTPResponse.parsers=htmlParser wmlParse
 
 # Define the HTML parser to be used.
 # Default parser:
-#htmlParser.className=org.apache.jmeter.protocol.http.parser.HtmlParserHTMLParser
+# This new parser (since 2.10) should perform better than all others
+# see https://issues.apache.org/bugzilla/show_bug.cgi?id=55632
+#htmlParser.className=org.apache.jmeter.protocol.http.parser.LagartoBasedHtmlParser
+
 # Other parsers:
+# Default parser before 2.10
+#htmlParser.className=org.apache.jmeter.protocol.http.parser.HtmlParserHTMLParser
 #htmlParser.className=org.apache.jmeter.protocol.http.parser.JTidyHTMLParser
 # Note that Regexp extractor may detect references that have been commented out.
 # In many cases it will work OK, but you should be aware that it may generate 
 # additional references.
 #htmlParser.className=org.apache.jmeter.protocol.http.parser.RegexpHTMLParser
-# This new parser (since 2.10) should perform better than all others
-# see https://issues.apache.org/bugzilla/show_bug.cgi?id=55632
-#htmlParser.className=org.apache.jmeter.protocol.http.parser.LagartoBasedHtmlParser
-# This parser is based on JSoup
+# This parser is based on JSoup, it should be the most accurate but less performant
+# than LagartoBasedHtmlParser
 #htmlParser.className=org.apache.jmeter.protocol.http.parser.RegexpHTMLParser
 
 htmlParser.types=text/html application/xhtml+xml application/xml text/xml

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java?rev=1530078&r1=1530077&r2=1530078&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java
(original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParser.java
Mon Oct  7 21:40:50 2013
@@ -67,7 +67,7 @@ public abstract class HTMLParser {
     public static final String PARSER_CLASSNAME = "htmlParser.className"; // $NON-NLS-1$
 
     public static final String DEFAULT_PARSER =
-        "org.apache.jmeter.protocol.http.parser.HtmlParserHTMLParser"; // $NON-NLS-1$
+        "org.apache.jmeter.protocol.http.parser.LagartoBasedHtmlParser"; // $NON-NLS-1$
 
     /**
      * Protected constructor to prevent instantiation except from within



Mime
View raw message