Skip to content

Comments

Dockerfile#2061

Merged
zachmu merged 25 commits intomainfrom
zachmu/docker2
Dec 4, 2025
Merged

Dockerfile#2061
zachmu merged 25 commits intomainfrom
zachmu/docker2

Conversation

@zachmu
Copy link
Member

@zachmu zachmu commented Dec 1, 2025 •
edited
Loading

Also: configure the super user and password with new env vars:

DOLTGRES_USER
DOLTGRES_PASSWORD

@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025 •
edited
Loading

Main PR
covering_index_scan_postgres 347.99/s 335.83/s -3.5%
index_join_postgres 158.43/s 157.76/s -0.5%
index_join_scan_postgres 187.40/s 184.52/s -1.6%
index_scan_postgres 11.65/s 11.63/s -0.2%
oltp_point_select 2227.67/s 2284.90/s +2.5%
oltp_read_only 1679.19/s 1727.74/s +2.8%
select_random_points 111.97/s 117.49/s +4.9%
select_random_ranges 125.82/s 128.85/s +2.4%
table_scan_postgres 11.00/s 11.08/s +0.7%
types_table_scan_postgres 5.16/s 5.15/s -0.2%

@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

Main PR
Total 42090 42090
Successful 17254 17254
Failures 24836 24836
Partial Successes1 5593 5593
Main PR
Successful 40.9931% 40.9931%
Failures 59.0069% 59.0069%

Footnotes

  1. These are tests that we're marking as Successful, however they do not match the expected output in some way. This is due to small differences, such as different wording on the error messages, or the column names being incorrect while the data itself is correct. ↩

@zachmu zachmu changed the title Zachmu/docker2 Dockerfile Dec 3, 2025
@zachmu zachmu requested a review from coffeegoddd December 3, 2025 01:56
@zachmu zachmu marked this pull request as ready for review December 3, 2025 01:56
coffeegoddd
coffeegoddd approved these changes Dec 3, 2025
View reviewed changes
Copy link
Contributor

@coffeegoddd coffeegoddd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just made a PR here with some tweaks.

Hydrocharged
Hydrocharged approved these changes Dec 4, 2025
View reviewed changes
Copy link
Collaborator

@Hydrocharged Hydrocharged left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of these changes LGTM!

@zachmu zachmu merged commit 67f8930 into main Dec 4, 2025
17 checks passed
@zachmu zachmu deleted the zachmu/docker2 branch December 4, 2025 16:58
@BrewTestBot BrewTestBot mentioned this pull request Dec 5, 2025
Merged
@roxblnfk roxblnfk mentioned this pull request Jan 23, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Hydrocharged Hydrocharged Hydrocharged approved these changes

@coffeegoddd coffeegoddd coffeegoddd approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zachmu @Hydrocharged @coffeegoddd