Discussion :

[doons]

Bonjour,

J'aimerais faire une requête qui permet de filtrer mes enregs selon l'année et le mois d'une date.
Pour cela j'ai fait la requête suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
@NamedQuery(name = "JourFerme.getJourFermeListeForSocieteAndYearMonth", 
		query = "SELECT jour FROM JourFerme jour WHERE jour.lieu = :lieu AND EXTRACT(YEAR, jour.date) = :annee AND EXTRACT(MONTH, jour.date) = :mois ORDER BY jour.date")

Mais lors du déploiement sur mon serveur glassfish, j'ai l'erreur suivante :

cannot Deploy test_erp
deploy is failing=Error occurred during deployment: Exception while deploying the app [test_erp] : Exception [EclipseLink-28019] (Eclipse Persistence Services - 2.5.2.v20140319-9ad6abd): org.eclipse.persistence.exceptions.EntityManagerSetupException
Exception Description: Deployment of PersistenceUnit [test_erp_PU] failed. Close all factories for this PersistenceUnit.
Internal Exception: Exception [EclipseLink-0] (Eclipse Persistence Services - 2.5.2.v20140319-9ad6abd): org.eclipse.persistence.exceptions.JPQLException
Exception Description: Syntax error parsing [SELECT jour FROM JourFerme jour WHERE jour.lieu = :lieu AND EXTRACT(YEAR, jour.date) = :annee AND EXTRACT(MONTH, jour.date) = :mois ORDER BY jour.date].
[60, 72] The right expression is not a valid expression.
[74, 150] The query contains a malformed ending.. Please see server.log for more details.

NB : J'utilise eclipseLink 2.5

Merci pour votre aide

[OButterlin]

En SQL standard, il faudrait utiliser YEAR(leChampDate) et MONTH(leChampDate)
EXTRACT est une fonction propre à MySQL et PostgreSQL... Tu as quoi comme base de données ?

[doons]

C'est une SQL Serveur.
J'ai vu qu'avec eclipselink il fallait utiliser extract() et pas seulement year() ou month()

[tchize_]

décomposer la date en base pour en extraire le mois et l'année n'est pas très perfrmant, ça ne facilie pas le travail du sgbd pour utiliser les index. Vu que tu veux tout ce qui se trouve dans un mois donné, cela veux dire que tu veux tout ce qui se trouve entre le début et la fin du mois. Refait ta requête dans ce sens, elle sera plus performante, plus facile à écrire, plus souple et indépendante du SGBD.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
 
@NamedQuery(name = "JourFerme.getJourFermeListeForSocieteInRange", 
		query = "SELECT jour FROM JourFerme jour WHERE jour.lieu = :lieu AND jour.date >= :minDate and jour.date <= :maxDate ORDER BY jour.date")

[doons]

J'hésitais entre faire une extraction et mettre une date de début et de fin.
Du coup je vais partir sur la date de début et de fin.

Merci les gars pour votre aide

[OButterlin]

Attention avec la solution de tchize_, si tu as un horodatage, il faudra positionner la partie heure/minute/seconde/milliseconde à 0 pour le début et à 23, 59, 59, 999 pour la fin...
Personnellement, je choisirais YEAR et MONTH...

[tchize_]

Year peut s'en sortir. Month va chier dans la colle sur les index, y a rien à y faire. Si t'as 500 donnée ok, mais quand tu travaille sur des millions de données, tu peux pas te permettre de laisser le sgbd scanner toute la table. pas de besoin de mettre à 59 59 999 sur la fin, suffit de mettre aussi à 0 et de remplacer le <= par un <. Tu peux aussi laisser le sgbd construire la date avec des fonction si tu veux pas t'emmerder avec ça. Le tout c'est d'appliquer les fonction sur tes paramètre et pas sur les colonne de la db.

[OButterlin]

pas de besoin de mettre à 59 59 999 sur la fin, suffit de mettre aussi à 0 et de remplacer le <= par un <.

Ben non, tu es obligé de mettre 23 59 59 999 sur la fin parce que si tu cibles une ligne correspondant à la date de fin mais contenant par exemple l'heure 10:25:06.102 tu ne l'auras pas dans ton résultat...

[tchize_]

Encore une fois non, 31 juillet 2016 10:25:06.102 est bien < 1er aout 2016 00:00:00

[OButterlin]

Oui, si tu prends le jour suivant ta date de fin, on est d'accord...
Mais ce n'est pas ce que tu avais mis dans ton post

Bref, on chipote mais dans le fond, on peut le faire... c'est tout ce qui compte

[doons]

J'utilise des dates sans horodatage (org.joda.time.LocalDate), du coup la solution début-fin va bien.

Merci pour ce complément :-)