+1 to put it in 6.0 Vladimir, one question : Why not use JCTools ( https://github.com/JCTools/JCTools) instead of Dexx HashMap?
Le lun. 30 mars 2020 à 14:05, Vladimir Sitnikov <[email protected]> a écrit : > >What do you mean by unsafe ? > > Suppose the property in question is TestElement.gui_class > If the code reads it as <<String > getPropertyAsString("TestElement.gui_class")>>, then the property can't be > modified, > so it is safe to use a shared property instance. > > However, if the client code does <<JMeterProperty prop = > getProperty("TestElement.gui_class");>> then > we have no idea what will happen next. The code might call > <<prop.setObjectValue(...)>> or do something else. > > That is why it is not safe to share the properties that were exposed > outside of AbstractTestElement class. > > >For me this should be 6.0 (even if it's backward compatible), > >it's a major change that needs to be highlighted. > > That is true. Async executor would indeed qualify as 6.0. > > Vladimir >
