Github user xguo27 commented on the pull request:

    https://github.com/apache/spark/pull/9553#issuecomment-160779029
  
    Hi @yhuai @liancheng:
    
    As I was hitting SPARK-11112 when testing my code, I rebased my branch and 
squashed my previous commits together. Now the new commit addresses the 
following point you brought up:
    
    1. call conf.getBoolean() to get the conf value at the right place
    2. using spark.sql.useHiveContext instead of spark.sql.hive.context
    3. using if/else instead of cases true/false
    4. provide extra information when logging
    
    Thanks for reviewing my code!


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