Skip to content

fix(sql): Partition pruning in deltas sql#146

Merged
devsjc merged 2 commits intomainfrom
devsjc/deltas-fix
Mar 31, 2026
Merged

fix(sql): Partition pruning in deltas sql#146
devsjc merged 2 commits intomainfrom
devsjc/deltas-fix

Conversation

@devsjc
Copy link
Copy Markdown
Contributor

@devsjc devsjc commented Mar 31, 2026

No description provided.

@github-actions
Copy link
Copy Markdown

Benchmark Results

Benchmark results
?   	github.com/openclimatefix/data-platform/cmd	[no test files]
?   	github.com/openclimatefix/data-platform/internal/gen/ocf/dp	[no test files]
?   	github.com/openclimatefix/data-platform/internal/interceptors	[no test files]
PASS
ok  	github.com/openclimatefix/data-platform/internal/server/dummy	0.005s
{"level":"debug","time":"2026-03-31T15:15:26Z","message":"Completed migrations"}
goos: linux
goarch: amd64
pkg: github.com/openclimatefix/data-platform/internal/server/postgres
cpu: Intel(R) Xeon(R) Platinum 8370C CPU @ 2.80GHz
BenchmarkPostgresClient/small/GetForecastAsTimeseries-4         	      62	  18070530 ns/op
BenchmarkPostgresClient/small/GetForecastAtTimestamp-4          	     308	   3721634 ns/op
BenchmarkPostgresClient/small/GetObservationsAsTimeseries-4     	    1129	   1016691 ns/op
BenchmarkPostgresClient/small/CreateForecast-4                  	      98	  11863633 ns/op
PASS
ok  	github.com/openclimatefix/data-platform/internal/server/postgres	64.517s
?   	github.com/openclimatefix/data-platform/internal/server/postgres/gen	[no test files]
Benchmark vs base branch
goos: linux
goarch: amd64
pkg: github.com/openclimatefix/data-platform/internal/server/postgres
cpu: Intel(R) Xeon(R) Platinum 8370C CPU @ 2.80GHz
                                                   │ bench-main.txt │     bench-devsjc-deltas-fix.txt      │
                                                   │     sec/op     │    sec/op     vs base                │
PostgresClient/small/GetForecastAsTimeseries-4         16.06m ± ∞ ¹   18.07m ± ∞ ¹       ~ (p=1.000 n=1) ²
PostgresClient/small/GetForecastAtTimestamp-4          3.655m ± ∞ ¹   3.722m ± ∞ ¹       ~ (p=1.000 n=1) ²
PostgresClient/small/GetObservationsAsTimeseries-4     1.020m ± ∞ ¹   1.017m ± ∞ ¹       ~ (p=1.000 n=1) ²
PostgresClient/small/CreateForecast-4                  10.88m ± ∞ ¹   11.86m ± ∞ ¹       ~ (p=1.000 n=1) ²
geomean                                                5.051m         5.337m        +5.65%
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

@devsjc devsjc merged commit f6cf2ee into main Mar 31, 2026
4 checks passed
@devsjc devsjc deleted the devsjc/deltas-fix branch March 31, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant