Discussion :

[cyberny]

Bonjour,

J'ai une requette qui fonctionne mais que je cherche à optimiser.

J'ai une table avec un id et une date pour chaque id non unique je peux avoir plusieurs date dont une eventuellement null

par exple :

id date
1 12/01/2009
1 08/09/2008
1 07/08/2008
2 12/04/2009
2
2 12/01/2009

lorsque je fais un

Code :

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

select max(date) from table group by id

j'obtiens toujours une date seulement je voudrais retourner null pour l'id 2

j'ai une solution hypra tordu mais je me dis qu'il y a plus simple c'est pour ca que je pose pas mon code, ca evite d'etre influencer dans une mauvaise voie.


Merci de votre aide
si je ne suis pas assez clair n'hesiter pas a me le dire.

Si il faut je posterai ma solution

Cordialement,

[Waldar]

Essayez ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
with test as
(
select 1 as id, to_date('12/01/2009', 'dd/mm/yyyy') as tdate from dual union all
select 1, to_date('08/09/2008', 'dd/mm/yyyy') from dual union all
select 1, to_date('07/08/2008', 'dd/mm/yyyy') from dual union all
select 2, to_date('12/04/2009', 'dd/mm/yyyy') from dual union all
select 2, null                                from dual union all
select 2, to_date('12/01/2009', 'dd/mm/yyyy') from dual
)
  SELECT id,
         nullif(max(nvl(tdate, date '9999-12-31')), date '9999-12-31') tdate_max
    FROM test
GROUP BY id
***************
ID	TDATE_MAX
1	12/01/2009
2

[cyberny]

Merci Waldar,
J'avais en fait la meme solution, mais j'utilisais un decode au lieu du nullif, ce qui facilite la lecture du code.
Par contre point de vue performance ca reste identique. mais j'ai vu que que le max seul me donne le meme cout à l'explain plan. Donc pas d'optimiation possible.
Merci pour l'aide.

[Waldar]

Juste pour information, nullif est rentré dans la norme SQL.
Mais comme vous j'utilise plus facilement un case when qui est plus facile en relecture.

[laurentschneider]

je ne suis pas un fan du codage en dur de la valeur bidon date '9999-12-31' car si un jour un développeur a la même idée alors la valeur '9999-12-31' sera remplacée par null...

je préferais quelque chose du style

Code :

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

max(d) keep (dense_rank last order by d)

mais bon, c'est sans doute moins performant

[Waldar]

Très bien vu, je n'y avais pas pensé (ce qui est malheureux avec la dernière entrée sur mon blog qui vante le mérite de ces fonctions).

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
WITH test AS
(
SELECT 1 AS id, to_date('12/01/2009', 'dd/mm/yyyy') AS tdate FROM dual union ALL
SELECT 1, to_date('08/09/2008', 'dd/mm/yyyy') FROM dual union ALL
SELECT 1, to_date('07/08/2008', 'dd/mm/yyyy') FROM dual union ALL
SELECT 2, to_date('12/04/2009', 'dd/mm/yyyy') FROM dual union ALL
SELECT 2, NULL                                FROM dual union ALL
SELECT 2, to_date('12/01/2009', 'dd/mm/yyyy') FROM dual
)
  SELECT DISTINCT
         id,
         max(tdate) keep (dense_rank last order by tdate asc nulls last) over(partition by id) tdate_max
    FROM test        
ORDER BY id asc

Sur une volumétrie de 6M de lignes, le group by a répondu en 11 secondes, le distinct + keep a répondu en 20 secondes.

C'est moins bon, mais sur des petites volumétries ça devrait passer relativement inaperçu.

[salim11]

Pourquoi le distinct??
Il faut le tester comme ça

Code :

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

1
2
3
4
5
 
SELECT      id,
         max(tdate) keep (dense_rank last ORDER BY tdate )  tdate_max
    FROM test   
GROUP BY id

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
WITH test AS
(
SELECT 1 AS id, to_date('12/01/2009', 'dd/mm/yyyy') AS tdate FROM dual union ALL
SELECT 1, to_date('08/09/2008', 'dd/mm/yyyy') FROM dual union ALL
SELECT 1, to_date('07/08/2008', 'dd/mm/yyyy') FROM dual union ALL
SELECT 2, to_date('12/04/2009', 'dd/mm/yyyy') FROM dual union ALL
SELECT 2, NULL                                FROM dual union ALL
SELECT 2, to_date('12/01/2009', 'dd/mm/yyyy') FROM dual
)
  SELECT      id,
         max(tdate) keep (dense_rank last ORDER BY tdate )  tdate_max
    FROM test   
group by id

[Waldar]

J'avais associé le keep avec le max analytique et non pas le max classique.
Grossière erreur de ma part !

Merci !