You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/09/22 02:50:25 UTC
svn commit: r578338 - in /jakarta/jmeter:
branches/rel-2-2/src/core/org/apache/jmeter/save/SaveService.java
trunk/src/core/org/apache/jmeter/save/SaveService.java
Author: sebb
Date: Fri Sep 21 17:50:24 2007
New Revision: 578338
URL: http://svn.apache.org/viewvc?rev=578338&view=rev
Log:
Disable SaveService version checks
Modified:
jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/save/SaveService.java
jakarta/jmeter/trunk/src/core/org/apache/jmeter/save/SaveService.java
Modified: jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/save/SaveService.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/save/SaveService.java?rev=578338&r1=578337&r2=578338&view=diff
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/save/SaveService.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/save/SaveService.java Fri Sep 21 17:50:24 2007
@@ -34,17 +34,17 @@
import java.nio.charset.Charset;
import org.apache.jmeter.samplers.SampleResult;
-import org.apache.jmeter.save.converters.BooleanPropertyConverter;
-import org.apache.jmeter.save.converters.HashTreeConverter;
-import org.apache.jmeter.save.converters.IntegerPropertyConverter;
-import org.apache.jmeter.save.converters.LongPropertyConverter;
-import org.apache.jmeter.save.converters.MultiPropertyConverter;
-import org.apache.jmeter.save.converters.SampleResultConverter;
-import org.apache.jmeter.save.converters.SampleSaveConfigurationConverter;
-import org.apache.jmeter.save.converters.StringPropertyConverter;
-import org.apache.jmeter.save.converters.TestElementConverter;
-import org.apache.jmeter.save.converters.TestElementPropertyConverter;
-import org.apache.jmeter.save.converters.TestResultWrapperConverter;
+//import org.apache.jmeter.save.converters.BooleanPropertyConverter;
+//import org.apache.jmeter.save.converters.HashTreeConverter;
+//import org.apache.jmeter.save.converters.IntegerPropertyConverter;
+//import org.apache.jmeter.save.converters.LongPropertyConverter;
+//import org.apache.jmeter.save.converters.MultiPropertyConverter;
+//import org.apache.jmeter.save.converters.SampleResultConverter;
+//import org.apache.jmeter.save.converters.SampleSaveConfigurationConverter;
+//import org.apache.jmeter.save.converters.StringPropertyConverter;
+//import org.apache.jmeter.save.converters.TestElementConverter;
+//import org.apache.jmeter.save.converters.TestElementPropertyConverter;
+//import org.apache.jmeter.save.converters.TestResultWrapperConverter;
import org.apache.jmeter.testelement.TestElement;
import org.apache.jmeter.util.JMeterUtils;
import org.apache.jorphan.collections.HashTree;
@@ -335,20 +335,20 @@
return rev;
}
- private static void checkVersion(Class clazz, String expected) {
-
- String actual = "*NONE*"; // $NON-NLS-1$
- try {
- actual = (String) clazz.getMethod("getVersion", null).invoke(null, null);
- actual = extractVersion(actual);
- } catch (Exception ignored) {
- // Not needed
- }
- if (0 != actual.compareTo(expected)) {
- versionsOK = false;
- log.warn("Version mismatch: expected '" + expected + "' found '" + actual + "' in " + clazz.getName());
- }
- }
+// private static void checkVersion(Class clazz, String expected) {
+//
+// String actual = "*NONE*"; // $NON-NLS-1$
+// try {
+// actual = (String) clazz.getMethod("getVersion", null).invoke(null, null);
+// actual = extractVersion(actual);
+// } catch (Exception ignored) {
+// // Not needed
+// }
+// if (0 != actual.compareTo(expected)) {
+// versionsOK = false;
+// log.warn("Version mismatch: expected '" + expected + "' found '" + actual + "' in " + clazz.getName());
+// }
+// }
// Routines for TestSaveService
static boolean checkPropertyVersion(){
@@ -361,28 +361,29 @@
static boolean checkVersions() {
versionsOK = true;
- checkVersion(BooleanPropertyConverter.class, "493779"); // $NON-NLS-1$
- checkVersion(HashTreeConverter.class, "514283"); // $NON-NLS-1$
- checkVersion(IntegerPropertyConverter.class, "493779"); // $NON-NLS-1$
- checkVersion(LongPropertyConverter.class, "493779"); // $NON-NLS-1$
- checkVersion(MultiPropertyConverter.class, "514283"); // $NON-NLS-1$
- checkVersion(SampleResultConverter.class, "571992"); // $NON-NLS-1$
-
- // Not built until later, so need to use this method:
- try {
- checkVersion(
- Class.forName("org.apache.jmeter.protocol.http.util.HTTPResultConverter"), // $NON-NLS-1$
- "514283"); // $NON-NLS-1$
- } catch (ClassNotFoundException e) {
- versionsOK = false;
- log.warn(e.getLocalizedMessage());
- }
- checkVersion(StringPropertyConverter.class, "493779"); // $NON-NLS-1$
- checkVersion(TestElementConverter.class, "549987"); // $NON-NLS-1$
- checkVersion(TestElementPropertyConverter.class, "549987"); // $NON-NLS-1$
- checkVersion(ScriptWrapperConverter.class, "514283"); // $NON-NLS-1$
- checkVersion(TestResultWrapperConverter.class, "514283"); // $NON-NLS-1$
- checkVersion(SampleSaveConfigurationConverter.class,"549936"); // $NON-NLS-1$
+ // Disable converter version checks as they are more of a nuisance than helpful
+// checkVersion(BooleanPropertyConverter.class, "493779"); // $NON-NLS-1$
+// checkVersion(HashTreeConverter.class, "514283"); // $NON-NLS-1$
+// checkVersion(IntegerPropertyConverter.class, "493779"); // $NON-NLS-1$
+// checkVersion(LongPropertyConverter.class, "493779"); // $NON-NLS-1$
+// checkVersion(MultiPropertyConverter.class, "514283"); // $NON-NLS-1$
+// checkVersion(SampleResultConverter.class, "571992"); // $NON-NLS-1$
+//
+// // Not built until later, so need to use this method:
+// try {
+// checkVersion(
+// Class.forName("org.apache.jmeter.protocol.http.util.HTTPResultConverter"), // $NON-NLS-1$
+// "514283"); // $NON-NLS-1$
+// } catch (ClassNotFoundException e) {
+// versionsOK = false;
+// log.warn(e.getLocalizedMessage());
+// }
+// checkVersion(StringPropertyConverter.class, "493779"); // $NON-NLS-1$
+// checkVersion(TestElementConverter.class, "549987"); // $NON-NLS-1$
+// checkVersion(TestElementPropertyConverter.class, "549987"); // $NON-NLS-1$
+// checkVersion(ScriptWrapperConverter.class, "514283"); // $NON-NLS-1$
+// checkVersion(TestResultWrapperConverter.class, "514283"); // $NON-NLS-1$
+// checkVersion(SampleSaveConfigurationConverter.class,"549936"); // $NON-NLS-1$
if (!PROPVERSION.equalsIgnoreCase(propertiesVersion)) {
log.warn("Bad _version - expected " + PROPVERSION + ", found " + propertiesVersion + ".");
Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/save/SaveService.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/save/SaveService.java?rev=578338&r1=578337&r2=578338&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/save/SaveService.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/save/SaveService.java Fri Sep 21 17:50:24 2007
@@ -34,17 +34,17 @@
import java.nio.charset.Charset;
import org.apache.jmeter.samplers.SampleResult;
-import org.apache.jmeter.save.converters.BooleanPropertyConverter;
-import org.apache.jmeter.save.converters.HashTreeConverter;
-import org.apache.jmeter.save.converters.IntegerPropertyConverter;
-import org.apache.jmeter.save.converters.LongPropertyConverter;
-import org.apache.jmeter.save.converters.MultiPropertyConverter;
-import org.apache.jmeter.save.converters.SampleResultConverter;
-import org.apache.jmeter.save.converters.SampleSaveConfigurationConverter;
-import org.apache.jmeter.save.converters.StringPropertyConverter;
-import org.apache.jmeter.save.converters.TestElementConverter;
-import org.apache.jmeter.save.converters.TestElementPropertyConverter;
-import org.apache.jmeter.save.converters.TestResultWrapperConverter;
+//import org.apache.jmeter.save.converters.BooleanPropertyConverter;
+//import org.apache.jmeter.save.converters.HashTreeConverter;
+//import org.apache.jmeter.save.converters.IntegerPropertyConverter;
+//import org.apache.jmeter.save.converters.LongPropertyConverter;
+//import org.apache.jmeter.save.converters.MultiPropertyConverter;
+//import org.apache.jmeter.save.converters.SampleResultConverter;
+//import org.apache.jmeter.save.converters.SampleSaveConfigurationConverter;
+//import org.apache.jmeter.save.converters.StringPropertyConverter;
+//import org.apache.jmeter.save.converters.TestElementConverter;
+//import org.apache.jmeter.save.converters.TestElementPropertyConverter;
+//import org.apache.jmeter.save.converters.TestResultWrapperConverter;
import org.apache.jmeter.testelement.TestElement;
import org.apache.jmeter.util.JMeterUtils;
import org.apache.jorphan.collections.HashTree;
@@ -350,20 +350,20 @@
return rev;
}
- private static void checkVersion(Class clazz, String expected) {
-
- String actual = "*NONE*"; // $NON-NLS-1$
- try {
- actual = (String) clazz.getMethod("getVersion", null).invoke(null, null);
- actual = extractVersion(actual);
- } catch (Exception ignored) {
- // Not needed
- }
- if (0 != actual.compareTo(expected)) {
- versionsOK = false;
- log.warn("Version mismatch: expected '" + expected + "' found '" + actual + "' in " + clazz.getName());
- }
- }
+// private static void checkVersion(Class clazz, String expected) {
+//
+// String actual = "*NONE*"; // $NON-NLS-1$
+// try {
+// actual = (String) clazz.getMethod("getVersion", null).invoke(null, null);
+// actual = extractVersion(actual);
+// } catch (Exception ignored) {
+// // Not needed
+// }
+// if (0 != actual.compareTo(expected)) {
+// versionsOK = false;
+// log.warn("Version mismatch: expected '" + expected + "' found '" + actual + "' in " + clazz.getName());
+// }
+// }
// Routines for TestSaveService
static boolean checkPropertyVersion(){
@@ -376,28 +376,29 @@
static boolean checkVersions() {
versionsOK = true;
- checkVersion(BooleanPropertyConverter.class, "493779"); // $NON-NLS-1$
- checkVersion(HashTreeConverter.class, "514283"); // $NON-NLS-1$
- checkVersion(IntegerPropertyConverter.class, "493779"); // $NON-NLS-1$
- checkVersion(LongPropertyConverter.class, "493779"); // $NON-NLS-1$
- checkVersion(MultiPropertyConverter.class, "514283"); // $NON-NLS-1$
- checkVersion(SampleResultConverter.class, "571992"); // $NON-NLS-1$
-
- // Not built until later, so need to use this method:
- try {
- checkVersion(
- Class.forName("org.apache.jmeter.protocol.http.util.HTTPResultConverter"), // $NON-NLS-1$
- "514283"); // $NON-NLS-1$
- } catch (ClassNotFoundException e) {
- versionsOK = false;
- log.warn(e.getLocalizedMessage());
- }
- checkVersion(StringPropertyConverter.class, "493779"); // $NON-NLS-1$
- checkVersion(TestElementConverter.class, "549987"); // $NON-NLS-1$
- checkVersion(TestElementPropertyConverter.class, "549987"); // $NON-NLS-1$
- checkVersion(ScriptWrapperConverter.class, "514283"); // $NON-NLS-1$
- checkVersion(TestResultWrapperConverter.class, "514283"); // $NON-NLS-1$
- checkVersion(SampleSaveConfigurationConverter.class,"549936"); // $NON-NLS-1$
+ // Disable converter version checks as they are more of a nuisance than helpful
+// checkVersion(BooleanPropertyConverter.class, "493779"); // $NON-NLS-1$
+// checkVersion(HashTreeConverter.class, "514283"); // $NON-NLS-1$
+// checkVersion(IntegerPropertyConverter.class, "493779"); // $NON-NLS-1$
+// checkVersion(LongPropertyConverter.class, "493779"); // $NON-NLS-1$
+// checkVersion(MultiPropertyConverter.class, "514283"); // $NON-NLS-1$
+// checkVersion(SampleResultConverter.class, "571992"); // $NON-NLS-1$
+//
+// // Not built until later, so need to use this method:
+// try {
+// checkVersion(
+// Class.forName("org.apache.jmeter.protocol.http.util.HTTPResultConverter"), // $NON-NLS-1$
+// "514283"); // $NON-NLS-1$
+// } catch (ClassNotFoundException e) {
+// versionsOK = false;
+// log.warn(e.getLocalizedMessage());
+// }
+// checkVersion(StringPropertyConverter.class, "493779"); // $NON-NLS-1$
+// checkVersion(TestElementConverter.class, "549987"); // $NON-NLS-1$
+// checkVersion(TestElementPropertyConverter.class, "549987"); // $NON-NLS-1$
+// checkVersion(ScriptWrapperConverter.class, "514283"); // $NON-NLS-1$
+// checkVersion(TestResultWrapperConverter.class, "514283"); // $NON-NLS-1$
+// checkVersion(SampleSaveConfigurationConverter.class,"549936"); // $NON-NLS-1$
if (!PROPVERSION.equalsIgnoreCase(propertiesVersion)) {
log.warn("Bad _version - expected " + PROPVERSION + ", found " + propertiesVersion + ".");
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org