-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make SMV easy to import and use #1489
Comments
That library looks like it could use a facelift but I guess is useful to look at.
One think that would make this ticket easier is if we moved all the arg parsing to python and out of the shell scripts.
Then the smv-blah scripts just become shebanged Python scripts, and importing via import becomes the real first class citizen since it’s used internally.
…On Nov 7, 2018, 16:30 -0800, Lane Barlow ***@***.***>, wrote:
Goal
I should be able to do the following
$ pip install smv
$ python
>>> import smv
>>> app = smvApp.createInstance([], None)
>>> app.runModule("foo")
without having to mess around and add things to my class path or sys.path
• The smv package should handle injecting the fat jar
• smv should try to add pyspark and py4j to the sys.path if they're not there (can consider https://github.com/minrk/findspark)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Needs a small whiteboard session I think. These are definitely interrelated goal. |
@laneb is the target to still use |
Not necessarily. I think jobs that need to run in yarn cluster mode will still have to be submitted via |
@jacobdr What's the goal that motivates removing the bash scripts? Just to simplify arg parsing? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Goal
I should be able to do the following
without having to mess around and add things to my class path or
sys.path
smv
package should handle injecting the fat jarsmv
should try to addpyspark
andpy4j
to thesys.path
if they're not there (can consider https://github.com/minrk/findspark)sys.dont_write_bytecode
instead of viaPYTHONDONTWRITEBYTECODE
env varThe text was updated successfully, but these errors were encountered: