From 86b741bb47862440150e4eaf6d726b755c8ff697 Mon Sep 17 00:00:00 2001 From: Michael Simons Date: Tue, 19 Jul 2016 23:40:48 +0200 Subject: [PATCH] Fix cumulativeEarnings As the graph suggests cumulative earnings are per store, not over all stores. --- .../main/java/org/jooq/example/chart/SakilaReportService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/jOOQ-examples/jOOQ-spark-chart-example/src/main/java/org/jooq/example/chart/SakilaReportService.java b/jOOQ-examples/jOOQ-spark-chart-example/src/main/java/org/jooq/example/chart/SakilaReportService.java index e6b170bf9f..2b2d89d058 100644 --- a/jOOQ-examples/jOOQ-spark-chart-example/src/main/java/org/jooq/example/chart/SakilaReportService.java +++ b/jOOQ-examples/jOOQ-spark-chart-example/src/main/java/org/jooq/example/chart/SakilaReportService.java @@ -139,7 +139,7 @@ public class SakilaReportService { .select( STORE.STORE_ID, PAYMENT.PAYMENT_DATE.cast(DATE).as(PAYMENT.PAYMENT_DATE), - sum(sum(PAYMENT.AMOUNT)).over(orderBy(PAYMENT.PAYMENT_DATE.cast(DATE))).as(PAYMENT.AMOUNT) + sum(sum(PAYMENT.AMOUNT)).over(partitionBy(STORE.STORE_ID).orderBy(PAYMENT.PAYMENT_DATE.cast(DATE))).as(PAYMENT.AMOUNT) ) .from(STORE) .join(INVENTORY).using(INVENTORY.STORE_ID)