view client/heapdumper/src/main/java/com/redhat/thermostat/client/heap/HeapView.java @ 656:3befd8ddad2f

Fix UI hangs caused by Memory Analyzer chart Certain operations in the "Memory Analyzer" tab can cause the entire GUI to free and the client application to hang. This behaviour can be seen if you leave thermostat (agent and client) running for a little while looking at a large java application (I used eclipse in my tests) and trigger heap dumps occasionally. It takes about 10 minutes for this bug to become fully visible. I noticed that the memory chart (that displays used/free memory) is recreated in HeapSwingView whenever HeapDumpController fetches some new data from storage. This creation also causes a clone of the TimeSeries objects containing the _entire_ dataset for the target application. A JFreeChart object is constructed from this but then thrown away after being used once. All this is done on the EDT too, which completely freezes the GUI. The client can only be killed by using kill -9 at this point. This commit changes the ChartPanel from a c.r.t.swing.ChartPanel to a org.jfree.chart.ChartPanel which automatically redraws itself whenever the underlying model changes. It also connects the ChartPanel directly to the model used by the controller so values are automatically updated in the chart without additional copying of memory. It is still possible to for the client to run out of memory and trigger OutOfMemoryError when triggering heap dumps, but at least the UI stays responsive enough and can be killed using ctrl-c on the console. Reviewed-by: jerboaa, neugens, vanaltj Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2012-October/003585.html
author Omair Majid <omajid@redhat.com>
date Thu, 04 Oct 2012 17:35:18 -0400
parents 0375b44c73d4
children
line wrap: on
line source

/*
 * Copyright 2012 Red Hat, Inc.
 *
 * This file is part of Thermostat.
 *
 * Thermostat is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published
 * by the Free Software Foundation; either version 2, or (at your
 * option) any later version.
 *
 * Thermostat is distributed in the hope that it will be useful, but
 * WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 * General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with Thermostat; see the file COPYING.  If not see
 * <http://www.gnu.org/licenses/>.
 *
 * Linking this code with other modules is making a combined work
 * based on this code.  Thus, the terms and conditions of the GNU
 * General Public License cover the whole combination.
 *
 * As a special exception, the copyright holders of this code give
 * you permission to link this code with independent modules to
 * produce an executable, regardless of the license terms of these
 * independent modules, and to copy and distribute the resulting
 * executable under terms of your choice, provided that you also
 * meet, for each linked independent module, the terms and conditions
 * of the license of that module.  An independent module is a module
 * which is not derived from or based on this code.  If you modify
 * this code, you may extend this exception to your version of the
 * library, but you are not obligated to do so.  If you do not wish
 * to do so, delete this exception statement from your version.
 */

package com.redhat.thermostat.client.heap;

import java.util.List;

import com.redhat.thermostat.client.heap.chart.OverviewChart;
import com.redhat.thermostat.client.osgi.service.BasicView;
import com.redhat.thermostat.common.ActionListener;
import com.redhat.thermostat.common.ActionNotifier;
import com.redhat.thermostat.common.heap.HeapDump;

public abstract class HeapView extends BasicView {
    
    public enum HeapDumperAction {
        DUMP_REQUESTED,
        ANALYSE,
        REQUEST_ABORTED
    }
   
    protected final ActionNotifier<HeapDumperAction> heapDumperNotifier;
    protected HeapView() {
        heapDumperNotifier = new ActionNotifier<HeapDumperAction>(this);
    }
    
    public void addDumperListener(ActionListener<HeapDumperAction> listener) {
        heapDumperNotifier.addActionListener(listener);
    }
    
    public void removeDumperListener(ActionListener<HeapDumperAction> listener) {
        heapDumperNotifier.removeActionListener(listener);
    }

    abstract public void updateUsedAndCapacity(String used, String capacity);
    /** View updates automatically based on the model */
    abstract public void setModel(OverviewChart model);
    abstract public void addHeapDump(HeapDump dump);
    abstract public void clearHeapDumpList();
    
    abstract public void openDumpView();
    abstract public void setChildView(BasicView childView);
    public abstract void notifyHeapDumpComplete();

    public abstract void updateHeapDumpList(List<HeapDump> heapDumps);

}