Skip to content

Make sure invalid reloptions are silently ignored when the caller req…#356

Merged
Ngalstyan4 merged 1 commit intomainfrom
narek/amoptions-validate
Nov 8, 2024
Merged

Make sure invalid reloptions are silently ignored when the caller req…#356
Ngalstyan4 merged 1 commit intomainfrom
narek/amoptions-validate

Conversation

@Ngalstyan4
Copy link
Copy Markdown
Contributor

…uests that

Per docs: https://www.postgresql.org/docs/current/index-functions.html

validate is false when loading options already stored in pg_catalog;
an invalid entry could only be found if the access method has changed its rules for options, and in that case ignoring obsolete entries is appropriate.

…uests that

Per docs: https://www.postgresql.org/docs/current/index-functions.html

> validate is false when loading options already stored in pg_catalog;
an invalid entry could only be found if the access method has changed
its rules for options, and in that case ignoring obsolete entries is
appropriate.
Copy link
Copy Markdown

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Hi! Looks like you've reached your API usage limit. You can increase it from your account settings page here: app.greptile.com/settings/usage

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 6, 2024

Benchmarks

metric old new pct change
recall (after create) 0.955 0.950 -0.52%
recall (after insert) 0.000 0.000 -
select tps 25838.402 28954.035 +12.06%
select bulk(100) tps 35.047 36.562 +4.32%
select latency (ms) 0.756 ± 1.518𝜎 0.728 ± 1.537𝜎 -3.70%
select bulk(100) latency (ms) 899.595 ± 111.774𝜎 854.789 ± 117.933𝜎 -4.98%
create latency (ms) 415422.390 413816.153 -0.39%
insert tps 466.930 429.678 -7.98%
insert bulk(100) tps 4.921 4.636 -5.81%
insert latency (ms) 67.928 ± 15.087𝜎 73.696 ± 17.383𝜎 +8.49%
insert bulk(100) latency (ms) 6374.399 ± 105.587𝜎 6767.100 ± 117.193𝜎 +6.16%
disk usage (bytes) 8192008192.000 8192008192.000 -

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@Ngalstyan4 Ngalstyan4 merged commit d414036 into main Nov 8, 2024
@Ngalstyan4 Ngalstyan4 deleted the narek/amoptions-validate branch November 8, 2024 16:23
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