Github user vanzin commented on the pull request:

    https://github.com/apache/spark/pull/4683#issuecomment-74977400
  
    Hi @zhzhan ,
    
    The reason why I haven't posted my code for review is because Yarn does not 
provide all the needed public APIs to implement this. If you read the 
discussion in SPARK-1537 you'll see what I mean. For example, this class is 
used by my code (and yours too) when it actually shouldn't be:
    
       @Singleton
        @Provider
        @Unstable
        @Private
        public class YarnJacksonJaxbJsonProvider extends 
JacksonJaxbJsonProvider {
    
    I strongly feel this code should not go in without Yarn providing the 
needed, *public* APIs to properly implement this.


---
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