You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2020/11/24 14:50:46 UTC
[jmeter] branch master updated: Bug 64935 - A broken plugin class
should not prevent JMeter from starting
This is an automated email from the ASF dual-hosted git repository.
pmouawad pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git
The following commit(s) were added to refs/heads/master by this push:
new 88be852 Bug 64935 - A broken plugin class should not prevent JMeter from starting
88be852 is described below
commit 88be852762cd5db86ecd7d8b035549a9782a5cb9
Author: pmouawad <p....@ubik-ingenierie.com>
AuthorDate: Tue Nov 24 15:49:49 2020 +0100
Bug 64935 - A broken plugin class should not prevent JMeter from
starting
---
.../src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java | 6 +++++-
xdocs/changes.xml | 1 +
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java b/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
index f05d262..0091776 100644
--- a/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
+++ b/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
@@ -232,8 +232,12 @@ public class JMeterMenuBar extends JMenuBar implements LocaleChangeListener {
MenuCreator creator = (MenuCreator) commandClass.getDeclaredConstructor().newInstance();
creators.add(creator);
}
+ } catch (NoClassDefFoundError e) {
+ log.error("Exception registering implementation:{} of interface:{}, a dependency used by the plugin class is missing",
+ strClassName, MenuCreator.class, e);
} catch (Exception e) {
- log.error("Exception registering {} with implementation: {}", MenuCreator.class, strClassName, e);
+ log.error("Exception registering implementation {} of interface:{}, a jar is probably missing",
+ strClassName, MenuCreator.class, e);
}
}
} catch (IOException e) {
diff --git a/xdocs/changes.xml b/xdocs/changes.xml
index 05381dc..5ba3735 100644
--- a/xdocs/changes.xml
+++ b/xdocs/changes.xml
@@ -227,6 +227,7 @@ applications when JMeter is starting up.</p>
<li><bug>64453</bug>Darklaf: Save Test Plan as New Folder failure</li>
<li><bug>64625</bug>Darklaf: trying to select a folder in Browse leads to an error popup and stacktrace</li>
<li><bug>64711</bug>Textarea Colors are not good in dark modes. Contributed by Jannis Weis</li>
+ <li><bug>64935</bug>A broken plugin class should not prevent JMeter from starting</li>
</ul>
<!-- =================== Thanks =================== -->
Re: [jmeter] branch master updated: Bug 64935 - A broken plugin class
should not prevent JMeter from starting
Posted by Philippe Mouawad <ph...@gmail.com>.
Ok by me if you want to change it
On Tue, Nov 24, 2020 at 3:57 PM Felix Schumacher <
felix.schumacher@internetallee.de> wrote:
>
> Am 24.11.20 um 15:50 schrieb pmouawad@apache.org:
> > This is an automated email from the ASF dual-hosted git repository.
> >
> > pmouawad pushed a commit to branch master
> > in repository https://gitbox.apache.org/repos/asf/jmeter.git
> >
> >
> > The following commit(s) were added to refs/heads/master by this push:
> > new 88be852 Bug 64935 - A broken plugin class should not prevent
> JMeter from starting
> > 88be852 is described below
> >
> > commit 88be852762cd5db86ecd7d8b035549a9782a5cb9
> > Author: pmouawad <p....@ubik-ingenierie.com>
> > AuthorDate: Tue Nov 24 15:49:49 2020 +0100
> >
> > Bug 64935 - A broken plugin class should not prevent JMeter from
> > starting
> > ---
> > .../src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java | 6
> +++++-
> > xdocs/changes.xml | 1
> +
> > 2 files changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git
> a/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
> b/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
> > index f05d262..0091776 100644
> > ---
> a/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
> > +++
> b/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
> > @@ -232,8 +232,12 @@ public class JMeterMenuBar extends JMenuBar
> implements LocaleChangeListener {
> > MenuCreator creator = (MenuCreator)
> commandClass.getDeclaredConstructor().newInstance();
> > creators.add(creator);
> > }
> > + } catch (NoClassDefFoundError e) {
> > + log.error("Exception registering implementation:{}
> of interface:{}, a dependency used by the plugin class is missing",
> > + strClassName, MenuCreator.class, e);
> > } catch (Exception e) {
> > - log.error("Exception registering {} with
> implementation: {}", MenuCreator.class, strClassName, e);
> > + log.error("Exception registering implementation
> {} of interface:{}, a jar is probably missing",
> > + strClassName, MenuCreator.class, e);
>
> Should we really take a guess at the reason for the Exception?
>
> The messages are almost identical, except that the first has no spaces
> between the colon and the placeholders and the second one has one space
> instead of the colon (plus the guessed reasons). I think we should add a
> space after the colons and we might add parenthesis around the
> placeholders: ... implementation: [{}] of ...
>
> (Will do so in a moment ;))
>
> Felix
>
> > }
> > }
> > } catch (IOException e) {
> > diff --git a/xdocs/changes.xml b/xdocs/changes.xml
> > index 05381dc..5ba3735 100644
> > --- a/xdocs/changes.xml
> > +++ b/xdocs/changes.xml
> > @@ -227,6 +227,7 @@ applications when JMeter is starting up.</p>
> > <li><bug>64453</bug>Darklaf: Save Test Plan as New Folder
> failure</li>
> > <li><bug>64625</bug>Darklaf: trying to select a folder in Browse
> leads to an error popup and stacktrace</li>
> > <li><bug>64711</bug>Textarea Colors are not good in dark modes.
> Contributed by Jannis Weis</li>
> > + <li><bug>64935</bug>A broken plugin class should not prevent JMeter
> from starting</li>
> > </ul>
> >
> > <!-- =================== Thanks =================== -->
> >
>
--
Cordialement.
Philippe Mouawad.
Re: [jmeter] branch master updated: Bug 64935 - A broken plugin class
should not prevent JMeter from starting
Posted by Felix Schumacher <fe...@internetallee.de>.
Am 24.11.20 um 15:50 schrieb pmouawad@apache.org:
> This is an automated email from the ASF dual-hosted git repository.
>
> pmouawad pushed a commit to branch master
> in repository https://gitbox.apache.org/repos/asf/jmeter.git
>
>
> The following commit(s) were added to refs/heads/master by this push:
> new 88be852 Bug 64935 - A broken plugin class should not prevent JMeter from starting
> 88be852 is described below
>
> commit 88be852762cd5db86ecd7d8b035549a9782a5cb9
> Author: pmouawad <p....@ubik-ingenierie.com>
> AuthorDate: Tue Nov 24 15:49:49 2020 +0100
>
> Bug 64935 - A broken plugin class should not prevent JMeter from
> starting
> ---
> .../src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java | 6 +++++-
> xdocs/changes.xml | 1 +
> 2 files changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java b/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
> index f05d262..0091776 100644
> --- a/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
> +++ b/src/core/src/main/java/org/apache/jmeter/gui/util/JMeterMenuBar.java
> @@ -232,8 +232,12 @@ public class JMeterMenuBar extends JMenuBar implements LocaleChangeListener {
> MenuCreator creator = (MenuCreator) commandClass.getDeclaredConstructor().newInstance();
> creators.add(creator);
> }
> + } catch (NoClassDefFoundError e) {
> + log.error("Exception registering implementation:{} of interface:{}, a dependency used by the plugin class is missing",
> + strClassName, MenuCreator.class, e);
> } catch (Exception e) {
> - log.error("Exception registering {} with implementation: {}", MenuCreator.class, strClassName, e);
> + log.error("Exception registering implementation {} of interface:{}, a jar is probably missing",
> + strClassName, MenuCreator.class, e);
Should we really take a guess at the reason for the Exception?
The messages are almost identical, except that the first has no spaces
between the colon and the placeholders and the second one has one space
instead of the colon (plus the guessed reasons). I think we should add a
space after the colons and we might add parenthesis around the
placeholders: ... implementation: [{}] of ...
(Will do so in a moment ;))
Felix
> }
> }
> } catch (IOException e) {
> diff --git a/xdocs/changes.xml b/xdocs/changes.xml
> index 05381dc..5ba3735 100644
> --- a/xdocs/changes.xml
> +++ b/xdocs/changes.xml
> @@ -227,6 +227,7 @@ applications when JMeter is starting up.</p>
> <li><bug>64453</bug>Darklaf: Save Test Plan as New Folder failure</li>
> <li><bug>64625</bug>Darklaf: trying to select a folder in Browse leads to an error popup and stacktrace</li>
> <li><bug>64711</bug>Textarea Colors are not good in dark modes. Contributed by Jannis Weis</li>
> + <li><bug>64935</bug>A broken plugin class should not prevent JMeter from starting</li>
> </ul>
>
> <!-- =================== Thanks =================== -->
>