Skip to content

Fixed spurious error message for event scheduling when the dolt_schem…#7336

Merged
zachmu merged 7 commits intomainfrom
zachmu/event-error
Jan 17, 2024
Merged

Fixed spurious error message for event scheduling when the dolt_schem…#7336
zachmu merged 7 commits intomainfrom
zachmu/event-error

Conversation

@zachmu
Copy link
Copy Markdown
Member

@zachmu zachmu commented Jan 17, 2024

…as table doesn't exist.

Copy link
Copy Markdown
Contributor

@fulghum fulghum left a comment

Choose a reason for hiding this comment

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

LGTM!

@coffeegoddd
Copy link
Copy Markdown
Contributor

@zachmu DOLT

comparing_percentages
99.997743 to 99.996126
version result total
ee0270d not ok 230
ee0270d ok 5937227
version total_tests
ee0270d 5937457
correctness_percentage
99.996126

@coffeegoddd
Copy link
Copy Markdown
Contributor

@coffeegoddd DOLT

comparing_percentages
99.997743 to 99.996126
version result total
4786c57 not ok 230
4786c57 ok 5937227
version total_tests
4786c57 5937457
correctness_percentage
99.996126

@coffeegoddd
Copy link
Copy Markdown
Contributor

@zachmu DOLT

comparing_percentages
99.997743 to 99.997743
version result total
8ac94d1 not ok 134
8ac94d1 ok 5937323
version total_tests
8ac94d1 5937457
correctness_percentage
99.997743

@coffeegoddd
Copy link
Copy Markdown
Contributor

@coffeegoddd DOLT

comparing_percentages
99.997743 to 99.997743
version result total
5f5202f not ok 134
5f5202f ok 5937323
version total_tests
5f5202f 5937457
correctness_percentage
99.997743

@zachmu zachmu merged commit ff1e8f7 into main Jan 17, 2024
@Hydrocharged Hydrocharged deleted the zachmu/event-error branch February 7, 2024 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants