forked from babelfish-for-postgresql/babelfish_extensions
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Testing community changes #54
Draft
ahmed-shameem
wants to merge
170
commits into
BABEL_3_4_STABLE
Choose a base branch
from
pg16-cherry-pick-bff-1
base: BABEL_3_4_STABLE
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,285
−6,053
Draft
Changes from 1 commit
Commits
Show all changes
170 commits
Select commit
Hold shift + click to select a range
ed77d64
Empty commit
ahmed-shameem 32ab138
Fix ATTIMEZONE test output
7c7c37b
Revert "Implement Supported Geospatial CASTs (#1945)"
ba87d48
Revert "Implemented TDS Receiver Support for Spatial Datatypes and Te…
4ea8031
Revert "Support GEOGRAPHY and GEOMETRY datatypes along with TDS Sende…
b001836
Disable postgis build in parallel query tests
379dd2c
Temporarily disable failing tests
4f87fdb
Temporarily disable failing parallel query tests
f892552
Temporarily disable failing parallel query tests (for real)
c6f5dfe
Disable BABEL-4281 from jdbc and table-variable from upgrade tests
ahmed-shameem 18cba75
Rerun tests
ahmed-shameem 0755989
Rerun tests
ahmed-shameem 3ba85f0
Rerun tests
ahmed-shameem 9495688
Rerun tests
ahmed-shameem f70cebd
Rerun tests.
f2f0371
Change internal RelFileNode references to RelFileNumber or RelFileLoc…
0734965
Rerun tests.
cdcb08d
Fix PIVOT to work with new node implementation.
96281b5
Rerun tests.
fbb7b6b
Update test and output related to PIVOT
ahmed-shameem 0e74717
Rerun tests.
606571d
Rerun tests.
d446013
Rerun tests.
24c0109
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
26fd430
Temporarily disable 'push' github workflows to prevent double work fo…
9e6d1fc
Remove obsoleted pltsql declarations
fdd2442
Disable pull request workflow for Babelfish Smoke Tests as well
52d7f70
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
14ac39b
Rerun tests
ahmed-shameem 181fcf6
Rerun tests
ahmed-shameem 5547849
Update hooks initialisation in extension
ahmed-shameem 43a9e6e
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
960b05b
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
f9a7624
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
c011cc4
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
a2156a0
Change use of opt_index_name to op_single_name in gram-tsql-rule.y.
9ca4f71
Temporarily disable BABEL-3293 due to plan change
3b2a130
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
6ef000e
Remove HAVE_UNIX_SOCKETS declaration as done in https://github.com/po…
ahmed-shameem 6d98e0d
Update test files and outputs which were affected by https://github.c…
ahmed-shameem 5712ff9
Rerun tests
ahmed-shameem 02f8383
Update ownership_restrictions_from_pg expected output because of http…
ahmed-shameem c541ead
Check pg_dump tests
ahmed-shameem d2fd0a4
Rerun tests
ahmed-shameem 11813b3
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
2d4e2e0
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
94ec3e5
Remove our artificial PG_SOMAXCONN limit on listen queue length.
61a848f
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
b2676d0
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
06022a6
Update opt field of GrantRoleStmt to NIL, previously it was admin_opt…
ahmed-shameem 3a8c41a
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
ahmed-shameem 1015c76
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
ahmed-shameem cf74bde
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
ahmed-shameem 19a7e16
Rerun tests
ahmed-shameem ade7b7f
Run tests for https://github.com/amazon-aurora/postgresql_modified_fo…
ahmed-shameem 77589f3
[Need to revert] Remove Werror to ignore warnings as error (revert af…
ahmed-shameem fdd3697
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
fc8e729
Revert "[Need to revert] Remove Werror to ignore warnings as error (r…
937743f
Update calls to pg_clean_ascii() to conform to new definition.
d1fdac2
Change import from guc.h to guc_hooks.h due to community commit 0a20ff5
7dae82e
Fix more includes
d2e113b
Fix build errors due to shadowed local variables, inline functions, a…
868e0ae
Various GUC fixes for (a73952b795632b2cf5acada8476e7cf75857e9be Add
e63f111
Update aclcheck and ownercheck function calls
ahmed-shameem 22c998e
Update Trigger object address lookup when it doesn't exist.
8869be9
Fix aclchecks in OBJECTPROPERTY()
9d2331c
Temporarily disable pgr_select test
267872f
Update test configurations to use major version upgrade for 15->lates…
a3d22cb
Temporarily disable 545_1 as it is hanging during parallel query mode…
9806559
Try fixing parallel query mode tests
123efc4
Temporarily disable 545_1 test (again) as it is hanging during parall…
98952c1
Temporarily disable BABEL-3092 in parallel query tests as well as it …
73bdc86
Temporarily disable BABEL-SEQUENCE test from parallel query mode test…
7e30c22
Temporarily disable BABEL-235 test due to changes to implicit_transac…
043809b
Temporarily disable TestSPPrepare test from parallel query mode tests…
fb2e12c
Temporarily disable sp_who tests from parallel query mode tests as it…
fa4d40d
Fix uses of GetSQLLocalTimestamp() and GetSQLCurrentTimestamp().
31bc572
Temporarily disable 'Minor Version Upgrade Tests for empty database' …
d0b5fc4
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
66f24a8
Fix use of requiredPerms in Babelfish
8252fc7
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
cc143db
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
651a95f
Further fix for RTE perms
66c5d62
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
1b55e9e
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
8223be6
Temporarily disable TestDatatypeAggSort due to plan cost change
c50b8ac
Disable TestDatatypeAggSort in the right place
df3203a
Rerun tests for https://github.com/amazon-aurora/postgresql_modified_…
f92c003
Ignore TestDatatypeAggSort in the right place (again).
391ae2a
Temporarily disable BABEL_COL_NAME test from upgrade and JDBC tests
3bdd019
Fix datetime funcs to use new DateTimeErrorExtra field
7eadac5
Fix varchar and json funcs to conform to new 'soft' error reporting f…
db25fee
Update lexer to use new decdigit symbol
71afb66
Reenable BABEL_COL_NAME test but temporarily disable the bad output.
41f7bb9
Update the following struct/functions to build extensions
ahmed-shameem 1903c7a
Fix build failure
ahmed-shameem b4a7549
Include varatt.h in files where necessary
ahmed-shameem 892ba59
Automatically revoke new logins from sysadmin upon creation.
0861af8
Test some role changes
8780dc0
More testing to fix roles
2f8d71c
test with existing user model
ritanwar 0b7b270
Allow dbo to be member of roles
ritanwar dd5f4fe
Use db_owner during create/drop/alter of a user/role
ritanwar 00a84e4
provid createrole to db_owner
ritanwar 682e4ab
more fixes
ritanwar 4347be0
Fix is_empty_role
ritanwar a4f38c1
Temporarily ignore failing tests
d73a45a
Add upgrade script for 4.0.0
6e4205d
test warning for upgrade logs
eba41ba
test
fdd4c22
Test
a193e99
Set current user to db_owner during DROP ROLE
4764978
Fix build error
28d54d9
Use DB name to find owner role name as db_owner in the catalog is not…
3f3c0f1
Typo
b7ac4d8
Add exception checking
8848246
Try fixing upgrade tests again
50a97da
Test
16bfa69
Use db_owner for CREATE LOGIN as well
e48564f
Revert "Use db_owner for CREATE LOGIN as well"
a244523
Also disable tests in the upgrade tests
ea776a7
Disable BABEL-4384 in upgrade tests
41234df
Fix 'errstart not called' error
ritanwar e194fa0
Use bootstrap superuser when creating/altering/dropping logins
40ff417
Actually give CREATEROLE to db owners
557de18
Use superuser in more cases for Babelfish role administration
43ae264
Use superuser during drop database to prevent errors after pg_dump/re…
5452d56
Use superuser when running createdb commands
e80ba26
Use superuser when dropping roles during drop db
9e883f3
Ignore BABEL-4279 in upgrade tests and use superuser to drop roles an…
821616f
Use superuser to CREATE/ALTER user/role
ritanwar 97f503f
Fix upgrade github action
ritanwar 252d91d
Fix tds_fdw init
ritanwar 2622895
Fix build
fca2c98
Revert "Fix uses of GetSQLLocalTimestamp() and GetSQLCurrentTimestamp…
e6a2559
Fix build for p16 branch point
cc47c89
Rerun tests
1baaf0b
Test
526501d
Rerun tests
44098fb
Properly check for collation case-insensitivity
17b72d4
Replace references to 'force_parallel_mode' to 'debug_parallel_query'
ae999d1
Fix expected output for ATTIMEZONE test
7313f21
Try fixing TAP tests
cc3fc1b
Use Babelfish SA instead of superuser
3c361fb
Typo
7eda9c3
More typos
2f91f70
Remove debug warning and check for babelfish SA during drop role as well
e61b78d
Disable BABEL-4168 test from 15.2 schedule
ritanwar 09b3551
Create a new role bbf_role_admin to administrate Babelfish roles.
b641bfc
Set createrole_grant_inherit to 'inherit' prior to creating new roles.
1cd3760
use pstrdup() for storing the old createrole_self_grant value
c1144b7
Grant bbf_role_admin membership with inherit true during upgrade
4693567
Add bbf_role_admin to sysadmin
e6ecf86
Fix upgrade
50d77f1
Check for bbf_role_admin instead of sa when dropping roles
462e214
Fix upgrade script
50a1496
debug logging for upgrade
f15c492
Rerun tests
67d5dca
Rerun tests
14eeeb5
Add bbf_role_admin to grantees as well during upgrade
3346f05
quick fix
2b25c48
Add bbf_role_admin to bbf catalog
ritanwar 9a4ad3f
Skip bbf_role_admin role
ritanwar 784d862
Fix test output
ritanwar 364a90d
quick fix
ritanwar bdf1df0
Remove DISTINCT for debugging
43c065f
Use DISTINCT for sp_helpuser
4b5cc3c
Revert "Use DISTINCT for sp_helpuser"
ritanwar 9d9ba51
Revert "Remove DISTINCT for debugging"
ritanwar 428d822
Revert test changes
ritanwar File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1635,11 +1635,11 @@ tsql_pivot_expr: TSQL_PIVOT '(' func_application FOR ColId IN_P in_expr ')' | |
List *l = list_make1(copyObject(n)); | ||
if (value_col_strlist == NULL || subsel_valuelists == NULL) | ||
{ | ||
value_col_strlist = list_make1(s->sval); | ||
value_col_strlist = list_make1(s); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is the main change - instead of using a List of char*, use a List of String nodes. Consumers of the list were updated to work with the new type. |
||
subsel_valuelists = list_make1(l); | ||
}else | ||
{ | ||
value_col_strlist = lappend(value_col_strlist, s->sval); | ||
value_col_strlist = lappend(value_col_strlist, s); | ||
subsel_valuelists = lappend(subsel_valuelists, l); | ||
} | ||
} | ||
|
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
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Somehow when a PIVOT is used within a view, this value is NULL, which would cause a segfault in the following line of code without this check. We should probably dig down to find out why this list is being passed as NULL for views.