[GitHub] [jmeter] vlsi opened a new pull request #676: Reduce memory consumption by the logging panel (disable undo events for it)

[GitBox]

vlsi opened a new pull request #676:
URL: https://github.com/apache/jmeter/pull/676


   ## Description
   
   `org.apache.jorphan.gui.ui.TextAreaUIWithUndo#createUI` is called by Swing 
automatically by ALL text area components (since JMeter installs 
   `TextAreaUIWithUndo` as a UI class).
   That is convenient since it makes all text fields and text areas to support 
undo transparently, however, it results in surprising memory consumption by the 
logging panel.
   
   Here's a heap dump:
   <img width="777" alt="dominator tree" 
src="https://user-images.githubusercontent.com/213894/140286033-baacc384-e7c2-467b-969a-671d12e3b6cf.png";>
   
   `TextComponentUI$1` is `UndoManager`. I moved the anonymous class to a 
top-level named class so it is easier to understand in the future.
   
   ## How Has This Been Tested?
   
   Test plan with JSR223 sampler that calls log.info("...") a lot.
   After the change, the undo manager and text area are no longer in the top 
consumers.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@jmeter.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org