changeset 3659:36edf9a71cbd

backend: Mention backend errors in importer. flys-backend/trunk@5253 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author Sascha L. Teichmann <sascha.teichmann@intevation.de>
date Thu, 23 Aug 2012 16:20:45 +0000
parents 55c4b7442236
children 976ead36192d
files flys-backend/ChangeLog flys-backend/doc/README.txt flys-backend/src/main/java/de/intevation/flys/importer/ImportRiver.java flys-backend/src/main/java/de/intevation/flys/importer/Importer.java flys-backend/src/main/java/de/intevation/flys/importer/parsers/HYKParser.java flys-backend/src/main/java/de/intevation/flys/importer/parsers/PRFParser.java
diffstat 6 files changed, 87 insertions(+), 21 deletions(-) [+]
line wrap: on
line diff
--- a/flys-backend/ChangeLog	Thu Aug 23 15:46:01 2012 +0000
+++ b/flys-backend/ChangeLog	Thu Aug 23 16:20:45 2012 +0000
@@ -1,3 +1,13 @@
+2012-08-23	Sascha L. Teichmann	<sascha.teichmann@intevation.de>
+
+	* doc/README.txt: Added error messages.
+
+	* src/main/java/de/intevation/flys/importer/Importer.java,
+	  src/main/java/de/intevation/flys/importer/parsers/PRFParser.java
+	  src/main/java/de/intevation/flys/importer/parsers/HYKParser.java
+	  src/main/java/de/intevation/flys/importer/ImportRiver.java:
+	  Made errors identifiable.
+
 2012-08-23	Sascha L. Teichmann	<sascha.teichmann@intevation.de>
 
 	* doc/README.txt: Documentation for the importer (german).
--- a/flys-backend/doc/README.txt	Thu Aug 23 15:46:01 2012 +0000
+++ b/flys-backend/doc/README.txt	Thu Aug 23 16:20:45 2012 +0000
@@ -89,12 +89,68 @@
 
 TODO SLT: Datenbank-Credentials dokumentieren.
 
-TODO SLT: Fehler und Warnungen aus separater Datei hier hin.
 
-Betrieb:
+Fehler und Warnungen:
+---------------------
+
+- 'error while parsing gew'
+  Die GEW-Datei ist fehlerhaft oder konnte nicht geöffnet werden.
+
+- 'File 'XYZ' is broken!'
+  Die Datei XYZ ist inkonsistent und führt zu Fehlern.
+
+- 'Error while parsing file for morph. width.'
+  Beim Lesen der morphologischen Breite trat ein Fehler auf.
+
+- 'Error while storing flow velocity model.'
+  Beim Schreiben eines Fliessgeschwindigkeitsmodells trat ein Fehler auf.
+
+- 'Error while storing flow velocity measurement.'
+  Beim Schreiben einer Fliessgeschwindigkeitsmessung trat ein Fehler auf. 
+
+- 'Error while storing sediment yield.'
+  Beim Schreiben einer Sedimentablagerung trat ein Fehler auf.
+
+- 'Error while storing waterlevel diff.'
+  Beim Schreiben einer Wassspiegeldifferenz trat ein Fehler auf.
+
+- 'Error while storing sq relation.'
+  Beim Schreiben einer S(Q) Beziehung trat ein Fehler auf.
+
+- 'Error reading PRF file.'
+  Beim Lesen einer PRF-Datei trat ein Fehler auf.
+
+- 'Error closing PRF file.'
+  Beim Schliessen einer PRF-Datei trat ein Fehler auf.
+
+- 'HYK 1: not enough elements in line #'
+- 'HYK 2: not enough elements in line #'
+- 'HYK 5: not enough elements in line #'
+- 'HYK 6: not enough elements in line #'
+  Eine Zeile in einer HYK-Datei hat nicht genügend Elemente.
+
+- 'HYK: parsing num zones, bottom or top height failed in line #'
+- 'HYK: HYK: number of flow zones mismatches in line #'
+  Die Anzahl der Zonen oder Daten über die Zonen sind nicht korrekt.
+
+- 'HYK: cannot parse number in line #'
+  Eine Zahl wurde erwartet.
+
+- 'HYK: Error reading file.'
+  Beim Lesen einer HYK-Datei trat ein Fehler auf.
+
+- 'HYK: Error closing file.'
+  Beim Schliessen einer HYK-Datei trat ein Fehler auf.
+  
+
+TODO SLT: Warnungen aus separater Datei hier hin.
+
+
+Betrie :
+--------
 
     Der Speicherverbrauch des Importers ist sehr hoch. Es ist empfehlenswert,
-    der JVM mindestens 8GiB Haupspeicher zuzuordnen: '-Xmx=8192m'
+    der JVM mindestens 8GiB Hauptspeicher zuzuordnen: '-Xmx8192m'
     Besonders speicherintensiv ist der Import der HYKs und der PRFs.
     Hier ist es unter Umständen empfehlenswert, diese in zwei oder drei
     Schritten zu importieren. Zuerst die sonstigen hydrologischen Daten
--- a/flys-backend/src/main/java/de/intevation/flys/importer/ImportRiver.java	Thu Aug 23 15:46:01 2012 +0000
+++ b/flys-backend/src/main/java/de/intevation/flys/importer/ImportRiver.java	Thu Aug 23 16:20:45 2012 +0000
@@ -1127,10 +1127,10 @@
                     width.storeDependencies(river);
                 }
                 catch (SQLException sqle) {
-                    log.error("Error while parsing file for morph. width.");
+                    log.error("Error while parsing file for morph. width.", sqle);
                 }
                 catch (ConstraintViolationException cve) {
-                    log.error("Error while parsing file for morph. width.");
+                    log.error("Error while parsing file for morph. width.", cve);
                 }
             }
         }
--- a/flys-backend/src/main/java/de/intevation/flys/importer/Importer.java	Thu Aug 23 15:46:01 2012 +0000
+++ b/flys-backend/src/main/java/de/intevation/flys/importer/Importer.java	Thu Aug 23 16:20:45 2012 +0000
@@ -123,7 +123,7 @@
                 infoGewParser.parse(new File(gew));
             }
             catch (IOException ioe) {
-                log.error("cannot while parsing: " + gew);
+                log.error("error while parsing gew: " + gew);
             }
         }
 
@@ -134,7 +134,7 @@
                 infoGewParser.parse(new File(gew));
             }
             catch (IOException ioe) {
-                log.error("cannot parse file: " + gew);
+                log.error("error while parsing gew: " + gew);
             }
         }
 
--- a/flys-backend/src/main/java/de/intevation/flys/importer/parsers/HYKParser.java	Thu Aug 23 15:46:01 2012 +0000
+++ b/flys-backend/src/main/java/de/intevation/flys/importer/parsers/HYKParser.java	Thu Aug 23 16:20:45 2012 +0000
@@ -124,7 +124,7 @@
                 switch (state) {
                     case LINE_1:
                         if (parts.length < 2) {
-                            log.error("1: not enough elements in line " +
+                            log.error("HYK 1: not enough elements in line " +
                                 in.getLineNumber());
                             return false;
                         }
@@ -163,7 +163,7 @@
 
                     case LINE_2:
                         if (parts.length < 3) {
-                            log.error("2: not enough elements in line " +
+                            log.error("HYK 2: not enough elements in line " +
                                 in.getLineNumber());
                             return false;
                         }
@@ -174,7 +174,7 @@
                         }
                         catch (NumberFormatException nfe) {
                             log.error(
-                                "parsing num zones, bottom or top height " +
+                                "HYK: parsing num zones, bottom or top height " +
                                 "failed in line " + in.getLineNumber());
                             return false;
                         }
@@ -189,7 +189,7 @@
                     case LINE_3:
                         if (parts.length != numZones) {
                             log.error(
-                                "number of flow zones mismatches " +
+                                "HYK: number of flow zones mismatches " +
                                 "in line " + in.getLineNumber());
                             return false;
                         }
@@ -211,7 +211,7 @@
                             }
                         }
                         catch (NumberFormatException nfe) {
-                            log.error("cannot parse number in line " +
+                            log.error("HYK: cannot parse number in line " +
                                 in.getLineNumber());
                             return false;
                         }
@@ -220,7 +220,7 @@
 
                     case LINE_5:
                         if (parts.length + coordPos < coords.length) {
-                            log.error("5: not enough elements in line " +
+                            log.error("HYK 5: not enough elements in line " +
                                 in.getLineNumber());
                             return false;
                         }
@@ -233,7 +233,7 @@
                             }
                         }
                         catch (NumberFormatException nfe) {
-                            log.error("cannot parse number in line " +
+                            log.error("HYK: cannot parse number in line " +
                                 in.getLineNumber());
                             return false;
                         }
@@ -241,7 +241,7 @@
                             BigDecimal a = coords[i];
                             BigDecimal b = coords[i == coords.length-1 ? i : i+1];
                             if (a.compareTo(b) > 0) {
-                                log.warn("zone coordinates swapped in line " +
+                                log.warn("HYK: zone coordinates swapped in line " +
                                     in.getLineNumber());
                                 BigDecimal c = a; a = b; b = c;
                             }
@@ -254,7 +254,7 @@
 
                     case LINE_6:
                         if (parts.length < 3) {
-                            log.error("6: not enough elements in line " +
+                            log.error("HYK 6: not enough elements in line " +
                                 in.getLineNumber());
                             return false;
                         }
@@ -264,7 +264,7 @@
                             distanceVR = new BigDecimal(parts[2]);
                         }
                         catch (NumberFormatException nfe) {
-                            log.error("cannot parse number in line " +
+                            log.error("HYK: cannot parse number in line " +
                                 in.getLineNumber());
                             return false;
                         }
@@ -281,7 +281,7 @@
             }
         }
         catch (IOException ioe) {
-            log.error(ioe);
+            log.error("HYK: Error reading file.", ioe);
             return false;
         }
         finally {
@@ -290,7 +290,7 @@
                     in.close();
                 }
                 catch (IOException ioe) {
-                    log.error(ioe);
+                    log.error("HYK: Error closing file.", ioe);
                 }
             }
         }
--- a/flys-backend/src/main/java/de/intevation/flys/importer/parsers/PRFParser.java	Thu Aug 23 15:46:01 2012 +0000
+++ b/flys-backend/src/main/java/de/intevation/flys/importer/parsers/PRFParser.java	Thu Aug 23 16:20:45 2012 +0000
@@ -368,7 +368,7 @@
             sortLists();
         }
         catch (IOException ioe) {
-            log.error(ioe);
+            log.error("Error reading PRF file.", ioe);
             return false;
         }
         finally {
@@ -377,7 +377,7 @@
                     in.close();
                 }
                 catch (IOException ioe) {
-                    log.error(ioe);
+                    log.error("Error closing PRF file.", ioe);
                 }
             }
         }

http://dive4elements.wald.intevation.org