Uh oh!
There was an error while loading. Please reload this page.
[SPARK-16253][SQL] make spark sql compatible with hive sql that using… - #14686
[SPARK-16253][SQL] make spark sql compatible with hive sql that using…#14686zenglinxi0615 wants to merge 1 commit into
Conversation
… python script transform like using 'xxx.py'
AmplabJenkins
commented
Aug 17, 2016
Can one of the admins verify this patch? |
| // for compatibility with hive | ||
| // if the script is a python/bash/... file, we should make it executable | ||
| if (new File(script).exists) { | ||
| new File(script).setExecutable(true) |
There was a problem hiding this comment.
Doesn't this create a huge security hole? Especially because we are not resetting this to its old state.
zenglinxi0615
commented
Aug 18, 2016
Have you tried it on spark 1.6.2? |
I know how to deal with it. the error is "Error: /bin/bash: test.py: command not found" so there is not bug and no need to change source code |
sorry for long time no response. |
HyukjinKwon
commented
May 11, 2017
@zenglinxi0615 Is it still active? I wonder if there is any argument against #14686 (comment). Otherwise, I would rather like to suggest to close this. |
## What changes were proposed in this pull request? This PR proposes to close PRs ... - inactive to the review comments more than a month - WIP and inactive more than a month - with Jenkins build failure but inactive more than a month - suggested to be closed and no comment against that - obviously looking inappropriate (e.g., Branch 0.5) To make sure, I left a comment for each PR about a week ago and I could not have a response back from the author in these PRs below: Closesapache#11129Closesapache#12085Closesapache#12162Closesapache#12419Closesapache#12420Closesapache#12491Closesapache#13762Closesapache#13837Closesapache#13851Closesapache#13881Closesapache#13891Closesapache#13959Closesapache#14091Closesapache#14481Closesapache#14547Closesapache#14557Closesapache#14686Closesapache#15594Closesapache#15652Closesapache#15850Closesapache#15914Closesapache#15918Closesapache#16285Closesapache#16389Closesapache#16652Closesapache#16743Closesapache#16893Closesapache#16975Closesapache#17001Closesapache#17088Closesapache#17119Closesapache#17272Closesapache#17971 Added: Closesapache#17778Closesapache#17303Closesapache#17872 ## How was this patch tested? N/A Author: hyukjinkwon <gurwls223@gmail.com> Closesapache#18017 from HyukjinKwon/close-inactive-prs.
What changes were proposed in this pull request?
make spark sql compatible with hive sql that using python script transform like using 'xxx.py'
How was this patch tested?
manual tests