Mercurial > hg > thermostat-ng > agent
view plugins/vm-classstat/agent/src/main/java/com/redhat/thermostat/vm/classstat/agent/internal/VmClassStatDAOImpl.java @ 2780:0633f63d464d
Remove typeadapters and use JSONService
This patch removes most of the type adapters from the agent, and
replaces them with the JSONService from thermostat-common.
The version of thermostat-common is updated to 0.1.2, which supports the
current JSON expected by the web-gateway
Reviewed-by: neugens
Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2017-October/025522.html
author | Simon Tooke <stooke@redhat.com> |
---|---|
date | Wed, 25 Oct 2017 12:53:20 -0400 |
parents | c30824312ec5 |
children |
line wrap: on
line source
/* * Copyright 2012-2017 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.vm.classstat.agent.internal; import java.net.URI; import java.util.Arrays; import java.util.logging.Level; import java.util.logging.Logger; import com.redhat.thermostat.agent.http.HttpRequestService; import com.redhat.thermostat.common.config.experimental.ConfigurationInfoSource; import com.redhat.thermostat.common.plugin.BackendConfigurationUtil; import com.redhat.thermostat.common.plugin.PluginDAOBase; import com.redhat.thermostat.common.plugin.SystemID; import com.redhat.thermostat.common.utils.LoggingUtils; import com.redhat.thermostat.lang.schema.JSONService; import com.redhat.thermostat.vm.classstat.model.VmClassStat; import org.apache.felix.scr.annotations.Activate; import org.apache.felix.scr.annotations.Component; import org.apache.felix.scr.annotations.ConfigurationPolicy; import org.apache.felix.scr.annotations.Deactivate; import org.apache.felix.scr.annotations.Modified; import org.apache.felix.scr.annotations.Reference; import org.apache.felix.scr.annotations.Service; @Component( policy = ConfigurationPolicy.REQUIRE, configurationPid = "vm-classstat" ) @Service(value = VmClassStatDAO.class) public class VmClassStatDAOImpl extends PluginDAOBase<VmClassStat> implements VmClassStatDAO { private static final Logger logger = LoggingUtils.getLogger(VmClassStatDAOImpl.class); private static final String PLUGIN_ID = "vm-classstat"; private final ConfigurationCreator configCreator; private BackendConfigurationUtil config; @Reference private ConfigurationInfoSource configurationInfoSource; @Reference private HttpRequestService httpRequestService; @Reference private JSONService jsonService; @Reference private SystemID systemID; public VmClassStatDAOImpl() { this(new ConfigurationCreator()); } VmClassStatDAOImpl(ConfigurationCreator creator) { this.configCreator = creator; } @Activate public void activate() throws Exception { config = configCreator.create(configurationInfoSource); } /* * Default OSGI behaviour is to restart the component when configuration changes. * The @modified method acts as a callback to prevent this from happening since * we don't need to restart the component for our use-case. */ @Modified protected void modified() { logger.config("Doing nothing on configuration change for plugin vm-classstat"); } @Override protected URI getPostURI(URI basepath, VmClassStat obj) { return basepath.resolve("systems/" + systemID.getSystemID() + "/jvms/" + obj.getJvmId()); } @Override protected String serialise(VmClassStat obj) { return jsonService.serialiase(Arrays.asList(obj)); } @Override protected HttpRequestService getHttpRequestService() { return httpRequestService; } @Override protected BackendConfigurationUtil getConfig() { return config; } protected void bindConfigurationInfoSource(ConfigurationInfoSource configurationInfoSource) { this.configurationInfoSource = configurationInfoSource; } protected void bindHttpRequestService(HttpRequestService httpRequestService) { this.httpRequestService = httpRequestService; } protected void unbindHttpRequestService(HttpRequestService httpRequestService) { this.httpRequestService = null; logger.log(Level.INFO, "Unbound HTTP service. Further attempts to store data will fail until bound again."); } protected void bindJsonService(JSONService jsonService) { this.jsonService = jsonService; } void bindSystemID(final SystemID id) { this.systemID = id; } protected Logger getLogger() { return logger; } // For Testing purposes static class ConfigurationCreator { BackendConfigurationUtil create(ConfigurationInfoSource source) { return new BackendConfigurationUtil(source, PLUGIN_ID); } } }