Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix generate matrix for olap for saturdays #10622

Merged
merged 1 commit into from
Feb 1, 2025

Conversation

Bodobolero
Copy link
Contributor

@Bodobolero Bodobolero commented Feb 1, 2025

Problem

when introducing pg17 for job step Generate matrix for OLAP benchmarks I introduced a syntax error that only hits on Saturdays.

Summary of changes

Remove trailing comma

successful test run

https://github.com/neondatabase/neon/actions/runs/13086363907

@Bodobolero Bodobolero requested a review from a team as a code owner February 1, 2025 07:15
Copy link

github-actions bot commented Feb 1, 2025

7414 tests run: 7062 passed, 0 failed, 352 skipped (full report)


Code coverage* (full report)

  • functions: 33.2% (8519 of 25632 functions)
  • lines: 49.0% (71436 of 145930 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
aa23aab at 2025-02-01T08:12:36.653Z :recycle:

@Bodobolero Bodobolero added this pull request to the merge queue Feb 1, 2025
Merged via the queue into main with commit b9e1a67 Feb 1, 2025
125 checks passed
@Bodobolero Bodobolero deleted the bodobolero/fix_benchmarking_saturday branch February 1, 2025 11:10
@Bodobolero Bodobolero removed the request for review from problame February 1, 2025 12:31
github-merge-queue bot pushed a commit that referenced this pull request Feb 3, 2025
## Problem

#10616 was only intended
temparily during the weekend, want to reset to prior state

## Summary of changes

revert #10616 but keep fixes in
#10622
winter-loo pushed a commit to winter-loo/neon that referenced this pull request Feb 4, 2025
## Problem

when introducing pg17 for job step `Generate matrix for OLAP benchmarks`
I introduced a syntax error that only hits on Saturdays.

## Summary of changes

Remove trailing comma

## successful test run

https://github.com/neondatabase/neon/actions/runs/13086363907
winter-loo pushed a commit to winter-loo/neon that referenced this pull request Feb 4, 2025
## Problem

neondatabase#10616 was only intended
temparily during the weekend, want to reset to prior state

## Summary of changes

revert neondatabase#10616 but keep fixes in
neondatabase#10622
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.

3 participants