agora inbox for pgsql-sql@postgresql.org  
help / color / mirror / Atom feed
From: Andreas Joseph Krogh <andreas@visena.com>
To: pgsql-sql@postgresql.org
Subject: Re: Problems with PARTITION BY with count() in window-func
Date: Wed, 8 Nov 2017 14:50:20 +0100 (CET)
Message-ID: <VisenaEmail.47.ec56caa56a402a62.15f9bdc4735@tc7-visena> (raw)
In-Reply-To: <VisenaEmail.2f.7083bea2d9429cac.15f9b162e38@tc7-visena>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-sql>

På onsdag 08. november 2017 kl. 11:17:39, skrev Andreas Joseph Krogh <
andreas@visena.com <mailto:andreas@visena.com>>:
Hi all.
 
I'm trying to count() all log-entries per activity per month in a separate 
column using count() over(partition by ...) but get an error I don't understand.
[snip]
select date_trunc('month', log.start_date::TIMESTAMP WITHOUT TIME ZONE) AS 
month , log.activity_id , count(log.entity_id) AS num_logs , count(log
.entity_id)OVER total_for_month_window AS total_for_month FROM log_entry log 
WHERE1 = 1 AND date_trunc('month', log.start_date::TIMESTAMP WITHOUT TIME ZONE) 
BETWEEN'2017-01-01'::DATE AND '2017-05-01'::DATE GROUP BY month, log.activity_id
WINDOWtotal_for_month_window AS (PARTITION BY date_trunc('month', log
.start_date::TIMESTAMP WITHOUT TIME ZONE)) ORDER BY date_trunc('month', log
.start_date::TIMESTAMP WITHOUT TIME ZONE) ASC ; 
 
Note the missing log.entity_id in GROUP BY. 
 
but this gives the error:
 
[42803] column "log.entity_id" must appear in the GROUP BY clause or be used 
in an aggregate function
 Details
 
This query gives the desired results:
 
SELECT DISTINCT date_trunc('month', log.start_date::TIMESTAMP WITHOUT TIME ZONE
)AS month , log.activity_id , count(log.entity_id) OVER(partition by date_trunc(
'month', log.start_date::TIMESTAMP WITHOUT TIME ZONE), log.activity_id) AS 
num_logs_per_activity ,count(log.entity_id) OVER (partition by date_trunc(
'month', log.start_date::TIMESTAMP WITHOUT TIME ZONE)) AS total_for_month FROM 
log_entrylog ORDER BY date_trunc('month', log.start_date::TIMESTAMP WITHOUT 
TIME ZONE) ASC, log.activity_id ASC ; 
month activity_id num_logs_per_activity total_for_month 2017-01-01 
00:00:00.000000 1 2 4 2017-01-01 00:00:00.000000 2 2 4 2017-02-01 
00:00:00.000000 1 5 6 2017-02-01 00:00:00.000000 2 1 6 
 
 
But I'd like a solution without the DISTINCT, if one exists?
 
If I introduce a new column, logged_for, and want to list number of logged 
entries per person per activity I can use this:
 
SELECT DISTINCT date_trunc('month', log.start_date::TIMESTAMP WITHOUT TIME ZONE
)AS month , log.logged_for , log.activity_id , count(log.entity_id) OVER(
partition bydate_trunc('month', log.start_date::TIMESTAMP WITHOUT TIME ZONE), 
log.logged_for, log.activity_id) AS num_logs_per_person_for_activity , count(log
.entity_id)OVER (partition by date_trunc('month', log.start_date::TIMESTAMP 
WITHOUT TIME ZONE), log.logged_for) AS total_for_person_for_month FROM log_entry
logORDER BY date_trunc('month', log.start_date::TIMESTAMP WITHOUT TIME ZONE) ASC
; 
Which gives:
month logged_for activity_id num_logs_per_person_for_activity 
total_for_person_for_month 2017-01-01 00:00:00.000000 5 1 2 4 2017-01-01 
00:00:00.000000 5 2 2 4 2017-01-01 00:00:00.000000 6 1 2 4 2017-01-01 
00:00:00.000000 6 2 2 4 2017-02-01 00:00:00.000000 5 1 5 6 2017-02-01 
00:00:00.000000 5 2 1 6 2017-02-01 00:00:00.000000 6 1 5 6 2017-02-01 
00:00:00.000000 6 2 1 6 
 
 
Is this the recommended way to do this, I mean - having "group by" in the 
"partition by" belonging to the OVER()-clause of the count-aggregates?
 
Thanks.
 
-- Andreas Joseph Krogh
CTO / Partner - Visena AS
Mobile: +47 909 56 963
andreas@visena.com <mailto:andreas@visena.com>
www.visena.com <https://www.visena.com;
 <https://www.visena.com;


 

view thread (3+ messages)  latest in thread

Message-ID: <VisenaEmail.47.ec56caa56a402a62.15f9bdc4735@tc7-visena>
Permalink:  ../VisenaEmail.47.ec56caa56a402a62.15f9bdc4735@tc7-visena/
Also on:    postgresql.org/message-id/VisenaEmail.47.ec56caa56a402a62.15f9bdc4735@tc7-visena

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-sql@postgresql.org
  Cc: andreas@visena.com
  Subject: Re: Problems with PARTITION BY with count() in window-func
  In-Reply-To: <VisenaEmail.47.ec56caa56a402a62.15f9bdc4735@tc7-visena>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox