JAL-3730 retry ping before failing; sysout logging changed to Cache.log

[jalview.git] / src / jalview / ext / ensembl / EnsemblSeqProxy.java

diff --git a/src/jalview/ext/ensembl/EnsemblSeqProxy.java b/src/jalview/ext/ensembl/EnsemblSeqProxy.java
index d483c3b..fe61a87 100644 (file)
--- a/src/jalview/ext/ensembl/EnsemblSeqProxy.java
+++ b/src/jalview/ext/ensembl/EnsemblSeqProxy.java
@@ -20,6 +20,17 @@
  */
 package jalview.ext.ensembl;
 
+import java.io.IOException;
+import java.net.MalformedURLException;
+import java.net.URL;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
+
+import org.json.simple.parser.ParseException;
+
 import jalview.analysis.AlignmentUtils;
 import jalview.analysis.Dna;
 import jalview.bin.Cache;
@@ -40,18 +51,6 @@ import jalview.util.Comparison;
 import jalview.util.DBRefUtils;
 import jalview.util.IntRangeComparator;
 import jalview.util.MapList;
-import jalview.util.Platform;
-
-import java.io.IOException;
-import java.net.MalformedURLException;
-import java.net.URL;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
-import java.util.List;
-import java.util.Map;
-
-import org.json.simple.parser.ParseException;
 
 /**
  * Base class for Ensembl sequence fetchers
@@ -211,8 +210,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
       EnsemblFeatures gffFetcher = new EnsemblFeatures(getDomain());
       EnsemblFeatureType[] features = getFeaturesToFetch();      
       
-      Platform.timeCheck("ESP.getsequencerec1", Platform.TIME_MARK);     
-
+      // Platform.timeCheck("ESP.getsequencerec1", Platform.TIME_MARK);
       
       AlignmentI geneFeatures = gffFetcher.getSequenceRecords(accId,
               features);
@@ -221,7 +219,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
         genomicSequence = geneFeatures.getSequenceAt(0);
       }
       
-      Platform.timeCheck("ESP.getsequencerec2", Platform.TIME_MARK);     
+      // Platform.timeCheck("ESP.getsequencerec2", Platform.TIME_MARK);
       
       if (genomicSequence != null)
       {
@@ -236,7 +234,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
            * fetch and map protein product, and add it as a cross-reference
            * of the retrieved sequence
            */
-            Platform.timeCheck("ESP.transferFeatures", Platform.TIME_MARK);      
+          // Platform.timeCheck("ESP.transferFeatures", Platform.TIME_MARK);
           addProteinProduct(querySeq);
         }
       }
@@ -245,7 +243,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
       System.err.println(
               "Error transferring Ensembl features: " + e.getMessage());
     }
-    Platform.timeCheck("ESP.addfeat done", Platform.TIME_MARK);          
+    // Platform.timeCheck("ESP.addfeat done", Platform.TIME_MARK);
   }
 
   /**
@@ -267,12 +265,12 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
     String accId = querySeq.getName();
     try
     {
-      System.out.println("Adding protein product for " + accId);
+      Cache.log.info("Adding protein product for " + accId);
       AlignmentI protein = new EnsemblProtein(getDomain())
               .getSequenceRecords(accId);
       if (protein == null || protein.getHeight() == 0)
       {
-        System.out.println("No protein product found for " + accId);
+        Cache.log.info("No protein product found for " + accId);
         return;
       }
       SequenceI proteinSeq = protein.getSequenceAt(0);
@@ -359,7 +357,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
   protected void getCrossReferences(SequenceI seq)
   {
          
-      Platform.timeCheck("ESP. getdataseq ", Platform.TIME_MARK);        
+    // Platform.timeCheck("ESP. getdataseq ", Platform.TIME_MARK);
 
          
     while (seq.getDatasetSequence() != null)
@@ -367,7 +365,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
       seq = seq.getDatasetSequence();
     }
 
-    Platform.timeCheck("ESP. getxref ", Platform.TIME_MARK);     
+    // Platform.timeCheck("ESP. getxref ", Platform.TIME_MARK);
 
     EnsemblXref xrefFetcher = new EnsemblXref(getDomain(), getDbSource(),
             getEnsemblDataVersion());
@@ -397,8 +395,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
 
     seq.addDBRef(self);
     
-    Platform.timeCheck("ESP. seqprox done ", Platform.TIME_MARK);        
-
+    // Platform.timeCheck("ESP. seqprox done ", Platform.TIME_MARK);
   }
 
   /**
@@ -419,7 +416,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
       inProgress = false;
       throw new JalviewException("ENSEMBL Rest API not available.");
     }
-       Platform.timeCheck("EnsemblSeqProx.fetchSeq ", Platform.TIME_MARK);
+    // Platform.timeCheck("EnsemblSeqProx.fetchSeq ", Platform.TIME_MARK);
 
     List<SequenceI> seqs = parseSequenceJson(ids);
     if (seqs == null)
@@ -432,7 +429,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
 
     if (seqs.size() != ids.size())
     {
-      System.out.println(String.format(
+      Cache.log.warn(String.format(
               "Only retrieved %d sequences for %d query strings",
               seqs.size(), ids.size()));
     }
@@ -486,8 +483,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
        * for now, assumes only one sequence returned; refactor if needed
        * in future to handle a JSONArray with more than one
        */
-       
-       Platform.timeCheck("ENS seqproxy", Platform.TIME_MARK);
+      // Platform.timeCheck("ENS seqproxy", Platform.TIME_MARK);
       Map<String, Object> val = (Map<String, Object>) getJSON(null, ids, -1, MODE_MAP, null);
       if (val == null)
          return null;
@@ -513,7 +509,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
       System.err.println("Error processing JSON response: " + e.toString());
       // ignore
     }
-       Platform.timeCheck("ENS seqproxy2", Platform.TIME_MARK);
+    // Platform.timeCheck("ENS seqproxy2", Platform.TIME_MARK);
     return result;
   }
 
@@ -661,7 +657,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
 
     if (regions.isEmpty())
     {
-      System.out.println("Failed to identify target sequence for " + accId
+      Cache.log.warn("Failed to identify target sequence for " + accId
               + " from genomic features");
       return null;
     }
@@ -826,9 +822,7 @@ public abstract class EnsemblSeqProxy extends EnsemblRestClient
       return false;
     }
 
-
-    Platform.timeCheck("ESP. xfer " + sfs.size(), Platform.TIME_MARK);   
-
+    // Platform.timeCheck("ESP. xfer " + sfs.size(), Platform.TIME_MARK);
 
     boolean result = transferFeatures(sfs, targetSequence, mapping,
             accessionId);