Github user MLnick commented on the pull request:

    https://github.com/apache/spark/pull/1916#issuecomment-52088220
  
    @kanzhang thanks for putting this together, and for the 1st draft of the 
blog post.
    
    @mateiz I'd say this is not necessary for 1.1 as it's intended to be a more 
advanced example rather than core code - however if the `Utils.classForName` is 
a bug or issue we should perhaps fix that in a separate PR before 1.1?
    
    I will go through the blog post and make some suggestions - from a quick 
read through it looks good though I agree with Matei that it could perhaps be 
made slightly more general.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to