-
Notifications
You must be signed in to change notification settings - Fork 5
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
root4j should not create new instances of the Hadoop Configuration #3
Comments
@superdude264 thanks for pointing out that this is indeed an issue - I actually didn't know what the right way of doing it is as you can see there is another version commented out... I will test it, however another issue could be (I have to check though - not sure) that HadoopConfiguration is not Serializable - have to check that VK |
Hadoop Configuration is not serializable - http://stackoverflow.com/questions/38224132/use-sparkcontext-hadoop-configuration-within-rdd-methods-closures-like-foreachp what can be done???
VK |
@vkhristenko any plans for adding support for this? XRootDConnector relies on this feature to set |
i will check what i can do... |
For some operations, root4j uses the same Hadoop Configuration instance as the one found in Spark, but in other cases it creates a new Hadoop Configuration, so it loses the configuration values set by Spark.
Here's an example of when it creates a new instance:
https://github.com/diana-hep/root4j/blob/master/src/main/java/org/dianahep/root4j/RootFileReader.java#L179
This issue was originally reported as a bug against the stocator program used by IBM Data Science Experience to read files Swift ("Object Storage service"). See: CODAIT/stocator#122
The text was updated successfully, but these errors were encountered: