1
74%
BABEL_5_X_DEV: 77%

Ran 01 Aug 2024 12:34PM UTC

Files 114

Run time 4s

Badge

Embed ▾

Committed 01 Aug 2024 11:52AM UTC coverage: 73.538% (+0.003%) from 73.535%

Job # 10197752006.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix behaviour of error message for GRANT (#2810) (#2818)

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 relevant tests for the same. After this change, Babelfish throws error during the parsing and gives syntax error.


Task: BABEL-5079

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

Run Details

42452 of 57728 relevant lines covered (73.54%)

110523.81 hits per line

babelfish-for-postgresql / babelfish_extensions / 10197752006 / 1
74%
BABEL_5_X_DEV: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10197752006.1

babelfish-for-postgresql / babelfish_extensions / 10197752006 / 1 74% BABEL_5_X_DEV: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10197752006.1

babelfish-for-postgresql / babelfish_extensions / 10197752006 / 1
74%
BABEL_5_X_DEV: 77%

README BADGES
x