Author: [log in to unmask]
Date: Thu Jul 7 18:16:50 2016
New Revision: 4422
Log:
[HPSJAVA-675] Minor cleanup
Modified:
java/trunk/crawler/src/main/java/org/hps/crawler/DatacatAddFile.java
java/trunk/datacat/src/main/java/org/hps/datacat/DataType.java
java/trunk/datacat/src/main/java/org/hps/datacat/DatacatConstants.java
java/trunk/datacat/src/main/java/org/hps/datacat/DatacatPrintRun.java
java/trunk/datacat/src/main/java/org/hps/datacat/DatacatUtilities.java
java/trunk/datacat/src/main/java/org/hps/datacat/FileEventRange.java
java/trunk/datacat/src/main/java/org/hps/datacat/FileFormat.java
java/trunk/datacat/src/main/java/org/hps/datacat/Site.java
java/trunk/run-database/src/main/java/org/hps/rundb/builder/BuilderCommandLine.java
java/trunk/run-database/src/main/java/org/hps/rundb/builder/DatacatBuilder.java
java/trunk/run-database/src/test/java/org/hps/rundb/builder/RunBuilderTest.java
Modified: java/trunk/crawler/src/main/java/org/hps/crawler/DatacatAddFile.java
=============================================================================
--- java/trunk/crawler/src/main/java/org/hps/crawler/DatacatAddFile.java (original)
+++ java/trunk/crawler/src/main/java/org/hps/crawler/DatacatAddFile.java Thu Jul 7 18:16:50 2016
@@ -10,6 +10,7 @@
import org.apache.commons.cli.Options;
import org.apache.commons.cli.ParseException;
import org.apache.commons.cli.PosixParser;
+import org.hps.datacat.DatacatConstants;
import org.hps.datacat.DatacatUtilities;
import org.hps.datacat.Site;
import org.srs.datacat.model.DatasetModel;
@@ -27,7 +28,7 @@
private String folder = null;
private Site site = Site.JLAB;
- private String datacatUrl = "http://hpsweb.jlab.org/datacat/r";
+ private String datacatUrl = DatacatConstants.DATACAT_URL;
private boolean dryRun = false;
/**
Modified: java/trunk/datacat/src/main/java/org/hps/datacat/DataType.java
=============================================================================
--- java/trunk/datacat/src/main/java/org/hps/datacat/DataType.java (original)
+++ java/trunk/datacat/src/main/java/org/hps/datacat/DataType.java Thu Jul 7 18:16:50 2016
@@ -3,7 +3,7 @@
/**
* Dataset types for HPS.
*
- * @author Jeremy McCormick, SLAC
+ * @author jeremym
*/
public enum DataType {
/**
Modified: java/trunk/datacat/src/main/java/org/hps/datacat/DatacatConstants.java
=============================================================================
--- java/trunk/datacat/src/main/java/org/hps/datacat/DatacatConstants.java (original)
+++ java/trunk/datacat/src/main/java/org/hps/datacat/DatacatConstants.java Thu Jul 7 18:16:50 2016
@@ -5,6 +5,7 @@
/**
* Static constants for use with the Data Catalog.
+ *
* @author jeremym
*/
public final class DatacatConstants {
Modified: java/trunk/datacat/src/main/java/org/hps/datacat/DatacatPrintRun.java
=============================================================================
--- java/trunk/datacat/src/main/java/org/hps/datacat/DatacatPrintRun.java (original)
+++ java/trunk/datacat/src/main/java/org/hps/datacat/DatacatPrintRun.java Thu Jul 7 18:16:50 2016
@@ -4,8 +4,6 @@
import java.util.SortedSet;
import java.util.TreeSet;
-import org.srs.datacat.client.Client;
-import org.srs.datacat.client.ClientBuilder;
import org.srs.datacat.model.DatasetModel;
import org.srs.datacat.model.DatasetResultSetModel;
import org.srs.datacat.model.dataset.DatasetLocationModel;
@@ -13,6 +11,7 @@
/**
* Example of printing information from all files for a given run in the datacat.
+ *
* @author jeremym
*/
public final class DatacatPrintRun {
@@ -26,20 +25,9 @@
}
private static void printRun(int run) throws Exception {
-
- /* initialize datacat client */
- Client client = new ClientBuilder().setUrl("http://hpsweb.jlab.org/datacat/r").build();
-
- /* perform dataset search */
- DatasetResultSetModel results = client.searchForDatasets(
- "/HPS/data/raw",
- "current", /* dataset version */
- "JLAB",
- "fileFormat eq 'EVIO' AND dataType eq 'RAW' AND runMin eq " + run, /* basic query */
- new String[] {"FILE"}, /* sort on file number */
- DatacatConstants.EVIO_METADATA /* metadata field values to return from query */
- );
-
+
+ DatasetResultSetModel results = DatacatUtilities.findEvioDatasets(run);
+
/* print results including metadata */
for (DatasetModel dataset : results) {
DatasetWithViewModel datasetView = (DatasetWithViewModel) dataset;
Modified: java/trunk/datacat/src/main/java/org/hps/datacat/DatacatUtilities.java
=============================================================================
--- java/trunk/datacat/src/main/java/org/hps/datacat/DatacatUtilities.java (original)
+++ java/trunk/datacat/src/main/java/org/hps/datacat/DatacatUtilities.java Thu Jul 7 18:16:50 2016
@@ -20,6 +20,7 @@
/**
* Data Catalog utility functions.
+ *
* @author jeremym
*/
public class DatacatUtilities {
Modified: java/trunk/datacat/src/main/java/org/hps/datacat/FileEventRange.java
=============================================================================
--- java/trunk/datacat/src/main/java/org/hps/datacat/FileEventRange.java (original)
+++ java/trunk/datacat/src/main/java/org/hps/datacat/FileEventRange.java Thu Jul 7 18:16:50 2016
@@ -11,6 +11,7 @@
/**
* Utility class for assocating a file in the datacat to its event ID range.
+ *
* @author jeremym
*/
public final class FileEventRange {
Modified: java/trunk/datacat/src/main/java/org/hps/datacat/FileFormat.java
=============================================================================
--- java/trunk/datacat/src/main/java/org/hps/datacat/FileFormat.java (original)
+++ java/trunk/datacat/src/main/java/org/hps/datacat/FileFormat.java Thu Jul 7 18:16:50 2016
@@ -7,7 +7,7 @@
/**
* Dataset file formats for HPS.
*
- * @author Jeremy McCormick, SLAC
+ * @author jeremym
*/
public enum FileFormat {
Modified: java/trunk/datacat/src/main/java/org/hps/datacat/Site.java
=============================================================================
--- java/trunk/datacat/src/main/java/org/hps/datacat/Site.java (original)
+++ java/trunk/datacat/src/main/java/org/hps/datacat/Site.java Thu Jul 7 18:16:50 2016
@@ -3,7 +3,7 @@
/**
* Site of a dataset (SLAC or JLAB).
*
- * @author Jeremy McCormick, SLAC
+ * @author jeremym
*/
public enum Site {
/**
Modified: java/trunk/run-database/src/main/java/org/hps/rundb/builder/BuilderCommandLine.java
=============================================================================
--- java/trunk/run-database/src/main/java/org/hps/rundb/builder/BuilderCommandLine.java (original)
+++ java/trunk/run-database/src/main/java/org/hps/rundb/builder/BuilderCommandLine.java Thu Jul 7 18:16:50 2016
@@ -12,14 +12,13 @@
import org.apache.commons.cli.ParseException;
import org.apache.commons.cli.PosixParser;
import org.hps.conditions.database.ConnectionParameters;
+import org.hps.datacat.DatacatConstants;
+import org.hps.datacat.Site;
import org.hps.rundb.DaoProvider;
import org.hps.rundb.RunManager;
import org.hps.rundb.RunSummaryDao;
import org.hps.rundb.RunSummaryImpl;
import org.srs.datacat.client.ClientBuilder;
-
-import org.hps.datacat.DatacatConstants;
-import org.hps.datacat.Site;
/**
* Creates a basic run database record from information in the data catalog
@@ -75,7 +74,7 @@
/**
* Data catalog site.
*/
- private String site = Site.JLAB.toString();
+ private Site site = Site.JLAB;
/**
* Data catalog URL.
@@ -136,7 +135,7 @@
// Site in the data catalog.
if (cl.hasOption("S")) {
- site = cl.getOptionValue("S");
+ site = Site.valueOf(cl.getOptionValue("S"));
}
// Set folder for dataset search.
Modified: java/trunk/run-database/src/main/java/org/hps/rundb/builder/DatacatBuilder.java
=============================================================================
--- java/trunk/run-database/src/main/java/org/hps/rundb/builder/DatacatBuilder.java (original)
+++ java/trunk/run-database/src/main/java/org/hps/rundb/builder/DatacatBuilder.java Thu Jul 7 18:16:50 2016
@@ -7,6 +7,8 @@
import java.util.logging.Level;
import java.util.logging.Logger;
+import org.hps.datacat.DatacatUtilities;
+import org.hps.datacat.Site;
import org.hps.record.triggerbank.TiTimeOffsetCalculator;
import org.srs.datacat.client.Client;
import org.srs.datacat.client.exception.DcClientException;
@@ -36,7 +38,7 @@
};
private Client datacatClient;
- private String site;
+ private Site site;
private String folder;
private List<File> files;
@@ -197,21 +199,15 @@
LOGGER.info("finding EVIO datasets for run " + getRun() + " in " + this.folder + " at " + this.site + " ...");
- DatasetResultSetModel results = datacatClient.searchForDatasets(
- this.folder,
- "current", /* dataset version */
- this.site,
- "fileFormat eq 'EVIO' AND dataType eq 'RAW' AND runMin eq " + getRun(), /* basic query */
- new String[] {"FILE"}, /* sort on file number */
- METADATA_FIELDS /* metadata field values to return from query */
- );
-
+ DatasetResultSetModel results = DatacatUtilities.findEvioDatasets(datacatClient, this.folder, this.site,
+ METADATA_FIELDS, new String[] {"FILE"}, getRun());
+
LOGGER.info("found " + results.getResults().size() + " EVIO datasets for run " + getRun());
return results;
}
- void setSite(String site) {
+ void setSite(Site site) {
this.site = site;
}
Modified: java/trunk/run-database/src/test/java/org/hps/rundb/builder/RunBuilderTest.java
=============================================================================
--- java/trunk/run-database/src/test/java/org/hps/rundb/builder/RunBuilderTest.java (original)
+++ java/trunk/run-database/src/test/java/org/hps/rundb/builder/RunBuilderTest.java Thu Jul 7 18:16:50 2016
@@ -6,6 +6,7 @@
import junit.framework.TestCase;
import org.hps.conditions.database.ConnectionParameters;
+import org.hps.datacat.Site;
import org.hps.rundb.RunManager;
import org.hps.rundb.RunSummaryImpl;
import org.srs.datacat.client.ClientBuilder;
@@ -16,7 +17,7 @@
private static String DATACAT_URL = "http://localhost:8080/datacat-v0.5-SNAPSHOT/r";
private static String SPREADSHEET = "/work/hps/rundb/HPS_Runs_2015_Sheet1.csv";
private static String FOLDER = "/HPS/test";
- private static String SITE = "SLAC";
+ private static Site SITE = Site.SLAC;
private static String EVIO_TEST_FILE = "/nfs/slac/g/hps3/data/engrun2015/evio/hps_005403.evio.0";
private static final ConnectionParameters CONNECTION_PARAMETERS =
|