Add logger and logging

- fix exception printing that goes nowhere
- make logger available for subclasses to use
This commit is contained in:
Tom Morris 2013-03-11 13:14:20 -04:00
parent a2a8f4af2e
commit 7578d3375f

View File

@ -1,128 +1,132 @@
/* /*
Copyright 2010, Google Inc. Copyright 2010, Google Inc.
All rights reserved. All rights reserved.
Redistribution and use in source and binary forms, with or without Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions are modification, are permitted provided that the following conditions are
met: met:
* Redistributions of source code must retain the above copyright * Redistributions of source code must retain the above copyright
notice, this list of conditions and the following disclaimer. notice, this list of conditions and the following disclaimer.
* Redistributions in binary form must reproduce the above * Redistributions in binary form must reproduce the above
copyright notice, this list of conditions and the following disclaimer copyright notice, this list of conditions and the following disclaimer
in the documentation and/or other materials provided with the in the documentation and/or other materials provided with the
distribution. distribution.
* Neither the name of Google Inc. nor the names of its * Neither the name of Google Inc. nor the names of its
contributors may be used to endorse or promote products derived from contributors may be used to endorse or promote products derived from
this software without specific prior written permission. this software without specific prior written permission.
THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
(INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/ */
package com.google.refine.model.recon; package com.google.refine.model.recon;
import java.io.Writer; import java.io.Writer;
import java.lang.reflect.Method; import java.lang.reflect.Method;
import java.util.HashMap; import java.util.HashMap;
import java.util.LinkedList; import java.util.LinkedList;
import java.util.List; import java.util.List;
import java.util.Map; import java.util.Map;
import java.util.Properties; import java.util.Properties;
import org.json.JSONException; import org.json.JSONException;
import org.json.JSONObject; import org.json.JSONObject;
import org.json.JSONWriter; import org.json.JSONWriter;
import org.slf4j.Logger;
import com.google.refine.Jsonizable; import org.slf4j.LoggerFactory;
import com.google.refine.model.Cell;
import com.google.refine.model.Project; import com.google.refine.Jsonizable;
import com.google.refine.model.Recon; import com.google.refine.model.Cell;
import com.google.refine.model.Row; import com.google.refine.model.Project;
import com.google.refine.model.Recon;
import edu.mit.simile.butterfly.ButterflyModule; import com.google.refine.model.Row;
abstract public class ReconConfig implements Jsonizable { import edu.mit.simile.butterfly.ButterflyModule;
static final public Map<String, List<Class<? extends ReconConfig>>> s_opNameToClass =
new HashMap<String, List<Class<? extends ReconConfig>>>(); abstract public class ReconConfig implements Jsonizable {
final static protected Logger LOGGER = LoggerFactory.getLogger("recon-config");
static final public Map<Class<? extends ReconConfig>, String> s_opClassToName =
new HashMap<Class<? extends ReconConfig>, String>(); static final public Map<String, List<Class<? extends ReconConfig>>> s_opNameToClass =
new HashMap<String, List<Class<? extends ReconConfig>>>();
static public void registerReconConfig(ButterflyModule module, String name, Class<? extends ReconConfig> klass) {
String key = module.getName() + "/" + name; static final public Map<Class<? extends ReconConfig>, String> s_opClassToName =
new HashMap<Class<? extends ReconConfig>, String>();
s_opClassToName.put(klass, key);
static public void registerReconConfig(ButterflyModule module, String name, Class<? extends ReconConfig> klass) {
List<Class<? extends ReconConfig>> classes = s_opNameToClass.get(key); String key = module.getName() + "/" + name;
if (classes == null) {
classes = new LinkedList<Class<? extends ReconConfig>>(); s_opClassToName.put(klass, key);
s_opNameToClass.put(key, classes);
} List<Class<? extends ReconConfig>> classes = s_opNameToClass.get(key);
classes.add(klass); if (classes == null) {
} classes = new LinkedList<Class<? extends ReconConfig>>();
s_opNameToClass.put(key, classes);
static public ReconConfig reconstruct(JSONObject obj) throws Exception { }
try { classes.add(klass);
String mode = obj.getString("mode"); }
// Backward compatibility static public ReconConfig reconstruct(JSONObject obj) throws Exception {
if ("extend".equals(mode) || "strict".equals(mode)) { try {
mode = "freebase/" + mode; String mode = obj.getString("mode");
} else if ("heuristic".equals(mode)) {
mode = "core/standard-service"; // legacy // Backward compatibility
} else if (!mode.contains("/")) { if ("extend".equals(mode) || "strict".equals(mode)) {
mode = "core/" + mode; mode = "freebase/" + mode;
} } else if ("heuristic".equals(mode)) {
mode = "core/standard-service"; // legacy
List<Class<? extends ReconConfig>> classes = s_opNameToClass.get(mode); } else if (!mode.contains("/")) {
if (classes != null && classes.size() > 0) { mode = "core/" + mode;
Class<? extends ReconConfig> klass = classes.get(classes.size() - 1); }
Method reconstruct = klass.getMethod("reconstruct", JSONObject.class); List<Class<? extends ReconConfig>> classes = s_opNameToClass.get(mode);
if (reconstruct != null) { if (classes != null && classes.size() > 0) {
return (ReconConfig) reconstruct.invoke(null, obj); Class<? extends ReconConfig> klass = classes.get(classes.size() - 1);
}
} Method reconstruct = klass.getMethod("reconstruct", JSONObject.class);
} catch (Exception e) { if (reconstruct != null) {
e.printStackTrace(); return (ReconConfig) reconstruct.invoke(null, obj);
} }
return null; }
} } catch (Exception e) {
LOGGER.error("Reconstruct failed",e);
abstract public int getBatchSize(); }
return null;
abstract public String getBriefDescription(Project project, String columnName); }
abstract public ReconJob createJob( abstract public int getBatchSize();
Project project,
int rowIndex, abstract public String getBriefDescription(Project project, String columnName);
Row row,
String columnName, abstract public ReconJob createJob(
Cell cell Project project,
); int rowIndex,
Row row,
abstract public List<Recon> batchRecon(List<ReconJob> jobs, long historyEntryID); String columnName,
Cell cell
abstract public Recon createNewRecon(long historyEntryID); );
public void save(Writer writer) { abstract public List<Recon> batchRecon(List<ReconJob> jobs, long historyEntryID);
JSONWriter jsonWriter = new JSONWriter(writer);
try { abstract public Recon createNewRecon(long historyEntryID);
write(jsonWriter, new Properties());
} catch (JSONException e) { public void save(Writer writer) {
e.printStackTrace(); JSONWriter jsonWriter = new JSONWriter(writer);
} try {
} write(jsonWriter, new Properties());
} } catch (JSONException e) {
LOGGER.error("Save failed",e);
}
}
}