1
76%
BABEL_5_X_DEV: 77%

Ran 31 Jul 2024 06:08PM UTC

Files 115

Run time 4s

Badge

Embed ▾

Committed 31 Jul 2024 05:20PM UTC coverage: 73.578% (+0.003%) from 73.575%

Job # 10184739814.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix behaviour of error message for GRANT (#2810)


Currently on executing queries like "GRANT EXECUTE ON PROCEDURE::test TO public;", babelfish throws error with error message 'GRANT ON PROCEDURE is not currently supported in Babelfish' which is incorrect as the above query has an incorrect syntax. With this change it now shows 'syntax error' for such cases and 'not currently supported' for allowed object types (according to GRANT documentation) which are not supported by us currently.

The issue arrised because the GRANT query is being successfully parsed
by our antlr parser and then the check for supported/unsupported occurs
resulting in the error being thrown in the
TsqlUnsupportedFeatureHandler() rather than the antlr parser.

In the rule definition of GRANT/REVOKE/DENY in TSqlParser.g4, object_type is used which contains some types like PROCEDURE, TABLE which are not syntactically correct according to the GRANT/REVOKE/DENY documentation. In this PR, I have created a new permission_object_type which contains only those types which are syntactically correct for GRANT/REVOKE/DENY and added relavent tests for the same. After this change, babelfish throws error during the parsing and gives syntax error.

Signed-off-by: P Aswini Kumar <aswinii@amazon.com>
Task: BABEL-5079

Run Details

43913 of 59682 relevant lines covered (73.58%)

125422.36 hits per line

babelfish-for-postgresql / babelfish_extensions / 10184739814 / 1
76%
BABEL_5_X_DEV: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10184739814.1

babelfish-for-postgresql / babelfish_extensions / 10184739814 / 1 76% BABEL_5_X_DEV: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10184739814.1

babelfish-for-postgresql / babelfish_extensions / 10184739814 / 1
76%
BABEL_5_X_DEV: 77%

README BADGES
x