-
Notifications
You must be signed in to change notification settings - Fork 0
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
f58d7073b7750d58b5d53232f0d6a967f902b78f^..db0a272d123b8d7f4d4acbeb54f27682a566be83 #65
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Reviewed-by: Amul Sul <[email protected]> Discussion: https://www.postgresql.org/message-id/flat/5b558da8-99fb-0a99-83dd-f72f05388517%40enterprisedb.com (cherry picked from commit f58d7073b7750d58b5d53232f0d6a967f902b78f)
A code comment said that the standard does not define a number for ERRCODE_SQL_JSON_ITEM_CANNOT_BE_CAST_TO_TARGET_TYPE, but this was fixed in a later draft version of the standard, so use that number now. (cherry picked from commit 976b06c6630cbf09bf933a006b5e662753e7c0be)
(cherry picked from commit 973137de0819793a82fc6dc91bddb78750380917)
Commit e3fcca0 reverted modifications to HOT for BRIN, but it also removed a couple unrelated tests from stats.sql. Reinstate those tests. Reported-by: Peter Eisentraut (cherry picked from commit 0df4eb3f708626ba3b0fdfab5afccf785e5e945c)
As $gzip is embedded in postgresql.conf \ needs to be escaped, otherwise guc.c will take it as a string escape. Similarly, if "$gzip" contains spaces, the prior incantation will fail. Both of these are common on windows. Reviewed-by: Peter Eisentraut <[email protected]> Discussion: https://postgr.es/m/[email protected] Backpatch: 15-, where the test was added in 027fa0f (cherry picked from commit a91242b1bcb312d6f482a758a432a32bd375896e)
This is in preparation for building postgres with meson / ninja. When building with meson, commands are run at the root of the build tree. Add an option to put build output into the appropriate place. This can be utilized by src/tools/msvc/ for a minor simplification, which also provides some coverage for the new option. To deal with dependencies to the variable set of input files to this script, add an option to generate a dependency file (which meson / ninja can consume). Reviewed-by: Peter Eisentraut <[email protected]> Discussion: https://postgr.es/m/[email protected] (cherry picked from commit 7c3c2cb9aeda4f288e89b25ede8cc2fe5997ac98)
This is in preparation for building postgres with meson / ninja. When building with meson, commands are run at the root of the build tree. Add an option to put build output into the appropriate place. This can be utilized by src/tools/msvc/ for a minor simplification, which also provides some coverage for the new option. Add option to generate a timestamp for check_rules.pl, so that proper dependencies on it having been run can be generated. Reviewed-by: Peter Eisentraut <[email protected]> Discussion: https://postgr.es/m/[email protected] (cherry picked from commit db0a272d123b8d7f4d4acbeb54f27682a566be83)
Sairakan
pushed a commit
to amazon-aurora/babelfish_extensions
that referenced
this pull request
Nov 30, 2023
Signed-off-by: Jason Teng <[email protected]>
2jungkook
approved these changes
Nov 30, 2023
anju15bharti
added a commit
that referenced
this pull request
Oct 25, 2024
Support dbcreator fixed server role #65
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
7 commits, no merge conflicts
Extension PR for validation: amazon-aurora/babelfish_extensions#54