[
https://issues.apache.org/jira/browse/HIVE-23078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17067794#comment-17067794
]
Naveen Gangam commented on HIVE-23078:
--------------------------------------
whoa .. this code has existed for over 7 years now via HIVE-2935 (part of some
intial core changes for the driver). Not clear what the original intent of it
was. My guess is to check if the application had write access on the file
system, maybe to be able to write some tmp files?
[~ashutoshc] You were one of the reviewers. Do you have any context to this?
Thanks
> Remove HiveDriver SecurityManager Check
> ---------------------------------------
>
> Key: HIVE-23078
> URL: https://issues.apache.org/jira/browse/HIVE-23078
> Project: Hive
> Issue Type: Improvement
> Components: JDBC
> Reporter: David Mollitor
> Assignee: David Mollitor
> Priority: Minor
> Attachments: HIVE-23078.1.patch
>
>
> {code:java|title=HiveDriver.java}
> public HiveDriver() {
> // TODO Auto-generated constructor stub
> SecurityManager security = System.getSecurityManager();
> if (security != null) {
> security.checkWrite("foobah");
> }
> }
> {code}
> Not sure why it needs to write a file called "foobah" but I checked out some
> other JDBC drivers and they do nothing like this. Remove this check; remove
> the constructor.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
