Open robverschoor opened 1 week ago
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.02% |
Covered Lines: | 42864 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.02% |
Covered Lines: | 42864 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.02% |
Covered Lines: | 42864 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.02% |
Covered Lines: | 42864 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.02% |
Covered Lines: | 42864 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.02% |
Covered Lines: | 42864 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.02% |
Covered Lines: | 42864 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.02% |
Covered Lines: | 42866 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
contrib/babelfishpg_tds/src/backend/tds/tdscomm.c | 3 | 73.33% | ||
<!-- | Total: | 3 | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.01% |
Covered Lines: | 42861 |
Relevant Lines: | 58538 |
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contrib/babelfishpg_tsql/src/tsqlIface.cpp | 55 | 57 | 96.49% | ||
<!-- | Total: | 76 | 78 | 97.44% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
contrib/babelfishpg_tds/src/backend/tds/tdscomm.c | 3 | 73.33% | ||
<!-- | Total: | 3 | --> |
Totals | |
---|---|
Change from base Build 9613188327: | 0.01% |
Covered Lines: | 42861 |
Relevant Lines: | 58538 |
Description
Cross-database references to the classic
sys*
catalogs (e.g.sysobjects
,syscolumns
,syslogins
, etc.), while specifying thedbo
schema or omitting the schema (e.g.db..sysobjects
notation), should not raise an error. For these 'classic' catalogs, referencing the object indbo
orsys
schema should be equivalent. This fix addresses the following notations which either raised an error or could return an incorrect value:Various existing test cases, which were raising an error previously, had to be adjusted for this fix.
Note: when the executing user has a non-
dbo
default schema, the case ofdbname..object
still ignores the default schema, but that is outside the scope of this fix (see BABEL-5050).Signed-off-by: Rob Verschoor rcv@amazon.com
Issues Resolved
BABEL-5041 Cross-DB system view query is not currently supported in Babelfish
Test Scenarios Covered
Use case based - Yes
Boundary conditions - N/A
Arbitrary inputs - N/A
Negative test cases - Yes
Minor version upgrade tests - N/A
Major version upgrade tests - N/A
Performance tests - N/A
Tooling impact - N/A
Client tests - N/A
Check List
By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.