[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15930549#comment-15930549 ] Vaibhav Gumashta commented on HIVE-12367: - Removing target 1.2.2 > Lock/unlock database should

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15217672#comment-15217672 ] Dapeng Sun commented on HIVE-12367: --- Thank [~ashutoshc] for your review and commit. > Lock/unlock

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15212685#comment-15212685 ] Hive QA commented on HIVE-12367: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15208331#comment-15208331 ] Ashutosh Chauhan commented on HIVE-12367: - +1 > Lock/unlock database should add current database

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059549#comment-15059549 ] Hive QA commented on HIVE-12367: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059574#comment-15059574 ] Dapeng Sun commented on HIVE-12367: --- Hi [~alangates], the failed cases (the same as

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15052093#comment-15052093 ] Dapeng Sun commented on HIVE-12367: --- Update patch with master. > Lock/unlock database should add

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15028284#comment-15028284 ] Dapeng Sun commented on HIVE-12367: --- Thank [~alangates] for your review :) > Lock/unlock database

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15022093#comment-15022093 ] Hive QA commented on HIVE-12367: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15023095#comment-15023095 ] Alan Gates commented on HIVE-12367: --- +1 > Lock/unlock database should add current database to inputs

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15021688#comment-15021688 ] Dapeng Sun commented on HIVE-12367: --- Hi [~alangates], I have updated the comment of DDL_NO_LOCK, I also

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018911#comment-15018911 ] Alan Gates commented on HIVE-12367: --- I agree that DDL_NO_LOCK is the right annotation, even on the lock

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15015432#comment-15015432 ] Dapeng Sun commented on HIVE-12367: --- I think the lock should still be {{DDL_NO_LOCK}}, I misunderstand

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014838#comment-15014838 ] Alan Gates commented on HIVE-12367: --- The test failures definitely look relevant and need to be fixed or

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014641#comment-15014641 ] Sergey Shelukhin commented on HIVE-12367: - Also, tests failed. Do files need the out file update?

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014639#comment-15014639 ] Sergey Shelukhin commented on HIVE-12367: - +0.9 from me. The patch makes sense, [~alangates] can

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15015083#comment-15015083 ] Dapeng Sun commented on HIVE-12367: --- Thank [~sershe] and [~alangates]. {quote}Do files need the out

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15003994#comment-15003994 ] Hive QA commented on HIVE-12367: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14996260#comment-14996260 ] Hive QA commented on HIVE-12367: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14996125#comment-14996125 ] Lefty Leverenz commented on HIVE-12367: --- [~dapengsun], the fix version shouldn't be set until the

[jira] [Commented] (HIVE-12367) Lock/unlock database should add current database to inputs and outputs of authz hook

[ https://issues.apache.org/jira/browse/HIVE-12367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14996169#comment-14996169 ] Dapeng Sun commented on HIVE-12367: --- [~leftylev], got it. Thank you for your information. :) >