Skip to content

/ adam

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

Jump to bottom

`_jvm.java.lang.Class.forName` does not work for certain configurations #1858

Closed
fnothaft opened this issue Jan 4, 2018 · 0 comments
Closed

`_jvm.java.lang.Class.forName` does not work for certain configurations #1858

fnothaft opened this issue Jan 4, 2018 · 0 comments
Assignees
@fnothaft
Labels
bug
Projects
Release 0.24.0
Milestone
0.24.0

Comments

@fnothaft
Copy link
Member

@fnothaft fnothaft commented Jan 4, 2018

This impacts the python transmute API. Specifically, transmute gets the converter between two types by calling sc._jvm.java.lang.Class.forName and constructing the class. This does not work for all py4j setups, but I'm not entirely sure why. However, doing:

getattr(sc._jvm, "path.to.my.class")

Does seem to work instead.
@fnothaft fnothaft added the bug label Jan 4, 2018
@fnothaft fnothaft added this to the 0.24.0 milestone Jan 4, 2018
@fnothaft fnothaft self-assigned this Jan 4, 2018
fnothaft pushed a commit to fnothaft/adam that referenced this issue Jan 4, 2018
Frank Austin Nothaft
[ADAM-1858] Use getattr instead of Class.forName from python API.
aa1888d 
Resolves bigdatagenomics#1858.
@fnothaft fnothaft mentioned this issue Jan 4, 2018
Merged
fnothaft pushed a commit to fnothaft/adam that referenced this issue Jan 9, 2018
Frank Austin Nothaft
[ADAM-1858] Use getattr instead of Class.forName from python API.
28bbdad 
Resolves bigdatagenomics#1858.
@heuermh heuermh added this to Triage in Release 0.24.0 Jan 9, 2018
@heuermh heuermh closed this in #1859 Jan 9, 2018
heuermh added a commit that referenced this issue Jan 9, 2018
Frank Austin Nothaft @heuermh
[ADAM-1858] Use getattr instead of Class.forName from python API.
71f9d13 
Resolves #1858.
@heuermh heuermh moved this from Triage to Completed in Release 0.24.0 Jan 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fnothaft fnothaft

Labels
bug
Projects
Release 0.24.0
Completed
Milestone
0.24.0
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@fnothaft