2010-03-29 5 views
7

Wir müssen sicherstellen, dass nur Ergebnisse innerhalb der letzten 30 Tage für eine JPQL-Abfrage zurückgegeben werden. Es folgt ein Beispiel:Wie führt man einen Zeitstempelvergleich mit der JPA-Abfrage durch?

Date now = new Date(); 
Timestamp thirtyDaysAgo = new Timestamp(now.getTime() - 86400000*30); 

Query query = em.createQuery(
    "SELECT msg FROM Message msg "+ 
    "WHERE msg.targetTime < CURRENT_TIMESTAMP AND msg.targetTime > {ts, '"+thirtyDaysAgo+"'}"); 
List result = query.getResultList();

Hier ist der Fehler die wir erhalten:

<openjpa-1.2.3-SNAPSHOT-r422266:907835 nonfatal user error> org.apache.openjpa.persistence.ArgumentException: An error occurred while parsing the query filter 'SELECT msg FROM BroadcastMessage msg WHERE msg.targetTime < CURRENT_TIMESTAMP AND msg.targetTime > {ts, '2010-04-18 04:15:37.827'}'. Error message: org.apache.openjpa.kernel.jpql.TokenMgrError: Lexical error at line 1, column 217. Encountered: "{" (123), after : ""

Hilfe!

+0

Was ist ts? Warum haben Sie Klammern in Ihrer Abfrage? {...} – Rick

+0

Es wird versucht, ein Zeitstempelliteral für die JDBC-Escape-Syntax zu verwenden, wie in den folgenden Links dokumentiert. Es sieht so aus, als wäre der Vorschlag von DataNucleus eine viel bessere Lösung. http://openjpa.apache.org/builds/latest/docs/manual/jpa_langref.html#jpa_langref_lit http://publib.boulder.ibm.com/infocenter/cscv/v10r1/topic/com.ibm.cloudscape.doc /rrefjdbc41784.html#rrefjdbc41784 – Lightbeard

+0

Sie sollten einen Fehler gegen OpenJPA öffnen, um den Code ... oder doc behoben zu bekommen. – Rick

Antwort

11

Die von Ihnen eingegebene Abfrage ist also nicht JPQL (was Sie anhand der JPA-Spezifikation sehen konnten). Wenn Sie ein Feld mit einem Datum vergleichen möchten, geben Sie das Datum als Parameter für die Abfrage ein

Dies ist NICHT SQL.

+4

Dies ist JPQL für unsere JPA-Implementierung, die hier dokumentiert ist: http://openjpa.apache.org/builds/latest/docs/manual/jpa_langref.html#jpa_langref_lit mit "JDBC Escape-Syntax". Sie sollten erwägen, etwas Sympathie für Leute zu haben, die versuchen, mit einer neuen Technologie anzufangen. – Lightbeard

+2

Ja, du hast recht; Ich habe vergessen, wie spezifisch RDBMS JPQL ist. Es wäre ratsam, eine solche Syntax zu vermeiden, da Sie gemäß der JPA-Spezifikation von dem JDBC-Treiber abhängig sind, der sie unterstützt, und die Abfrage ist mit einem Parameter viel sauberer. – DataNucleus

9

Die JDBC-Escape-Syntax wird in der von Ihnen verwendeten Version von OpenJPA möglicherweise nicht unterstützt. Die Dokumentation für die neueste Version 1.2.x ist hier: http://openjpa.apache.org/builds/1.2.2/apache-openjpa-1.2.2/docs/manual/manual.html#jpa_langref_lit.

Die Dokumentation bereits erwähnt bezieht sich auf die Dokumentation für OpenJPA 2.0.0 (spätestens): http://openjpa.apache.org/builds/latest/docs/manual/jpa_langref.html#jpa_langref_lit

Das heißt gibt es keinen Grund, warum Sie eine Zeichenfolge in Ihre JPQL injizieren wollen? Was ist mit etwas wie dem folgenden Ausschnitt?

Date now = new Date(); 
Date thirtyDaysAgo = new Date(now.getTime() - (30 * MS_IN_DAY)); 

Query q = em.createQuery("Select m from Message m " 
    + "where m.targetTime < :now and m.targetTime > :thirtyDays"); 
q.setParameter("now", now); 
q.setParameter("thirtyDays", thirtyDaysAgo); 

List<Message> results = (List<Message>) q.getResultList(); 
+4

Ich würde lieber den Parameter "jetzt" anstelle der JPQL-Funktion "current_timestamp" wie im DataNucleus-Vorschlag verwenden, da die Funktion die Zeit auf dem Datenbankserver zurückgibt und newDatum() die Zeit auf dem Java-Container zurückgibt. Wenn sie nicht die gleiche Maschine sind, kann dies zu Fehlern führen, wenn die Uhren nicht synchronisiert werden. –

+4

FYI, in JPQL können Sie eine TypedQuery verwenden, um Umwandlungen zu vermeiden. –