Skip to content

Implemented a couple jsonb functions#1243

Merged
zachmu merged 1 commit intomainfrom
zachmu/jsonb-funcs
Mar 6, 2025
Merged

Implemented a couple jsonb functions#1243
zachmu merged 1 commit intomainfrom
zachmu/jsonb-funcs

Conversation

@zachmu
Copy link
Copy Markdown
Member

@zachmu zachmu commented Mar 3, 2025

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 3, 2025

Main PR
covering_index_scan_postgres 347.00/s 319.46/s -8.0%
index_join_postgres 143.40/s 142.66/s -0.6%
index_join_scan_postgres 173.82/s 171.86/s -1.2%
index_scan_postgres 11.52/s 11.29/s -2.0%
oltp_point_select 2604.67/s 2514.38/s -3.5%
oltp_read_only 1740.32/s 1736.62/s -0.3%
select_random_points 100.19/s 100.67/s +0.4%
select_random_ranges 119.10/s 123.09/s +3.3%
table_scan_postgres 9.13/s 9.18/s +0.5%
types_table_scan_postgres 4.92/s 4.94/s +0.4%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 3, 2025

Main PR
Total 42090 42090
Successful 15686 15685
Failures 26404 26405
Partial Successes1 5209 5205
Main PR
Successful 37.2678% 37.2654%
Failures 62.7322% 62.7346%

${\color{lightgreen}Progressions (3)}$

jsonb

QUERY: SELECT jsonb_build_array('a', NULL);
QUERY: SELECT jsonb_build_object('a', NULL);
QUERY: SELECT jsonb_build_object(1,2);

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 requested a review from Hydrocharged March 3, 2025 23:36
@zachmu zachmu force-pushed the zachmu/jsonb-funcs branch 2 times, most recently from bc18de5 to 10f7a7d Compare March 4, 2025 01:23
Copy link
Copy Markdown
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.

LGTM!

@zachmu zachmu merged commit 015deee into main Mar 6, 2025
32 checks passed
@zachmu zachmu deleted the zachmu/jsonb-funcs branch March 6, 2025 01:37
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.

2 participants