Github user mateiz commented on the pull request:

    https://github.com/apache/spark/pull/999#issuecomment-46101589
  
    Also another note for @marmbrus as well: I'd really try to minimize the use 
of top-level SQLContext methods being called without `context.` in front of 
them due to `import SQLContext._`. This is confusing for newcomers to Scala and 
makes the code harder to translate across languages. You can *maybe* use it for 
`sql()` and `hql()` but even then I'd consider moving the implicit conversions 
somewhere else in a future version of the API (e.g. `import SQLContext._` to 
give you those). Let's discuss that more later, for now though it would be nice 
to update the examples to avoid these for `jsonFile`, `registerRDDAsTable`, etc.


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

Reply via email to