[
https://issues.apache.org/jira/browse/SENTRY-2066?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16261762#comment-16261762
]
Hadoop QA commented on SENTRY-2066:
-----------------------------------
Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12898732/SENTRY-2066.003.patch
against master.
{color:green}Overall:{color} +1 all checks pass
{color:green}SUCCESS:{color} all tests passed
Console output:
https://builds.apache.org/job/PreCommit-SENTRY-Build/3510/console
This message is automatically generated.
> DB name is not set for AlterTable
> ----------------------------------
>
> Key: SENTRY-2066
> URL: https://issues.apache.org/jira/browse/SENTRY-2066
> Project: Sentry
> Issue Type: Bug
> Components: Sentry
> Affects Versions: 2.0.0
> Reporter: Na Li
> Assignee: Na Li
> Priority: Critical
> Labels: 2.0
> Attachments: SENTRY-2066.001.patch, SENTRY-2066.002.patch,
> SENTRY-2066.003.patch
>
>
> SENTRY-1087 introduced a regression: the database name is not set in alter
> table column command.
> SENTRY-1087 added a new case below, and it causes the currDB not set because
> its children are "TOK_TABNAME" and "TOK_ALTERTABLE_RENAMECOL" NONE IS
> "TOK_ALTERTABLE_SERIALIZER"
> {code}
> case HiveParser.TOK_ALTERTABLE:
>
> for (Node childNode : ast.getChildren()) {
> ASTNode childASTNode = (ASTNode) childNode;
> if
> ("TOK_ALTERTABLE_SERIALIZER".equals(childASTNode.getText())) {
> ASTNode serdeNode = (ASTNode) childASTNode.getChild(0);
> String serdeClassName =
> BaseSemanticAnalyzer.unescapeSQLString(serdeNode.getText());
> setSerdeURI(serdeClassName);
> currDB = getCanonicalDb();
> }
> }
> break;
> {code}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
