Github user doanduyhai commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/80#issuecomment-111465199
  
    Hello @Leemoonsoo 
    
    So,
    
    1) I've renamed `displayAsTable()` to `display()` as suggested
    2) I've introduce a default limit to RDD ( `rdd.take()`) using the 
`zeppeline.spark.maxResult` property
    3) User can override the default limit to pass in their own limit value in 
the `display()` method
    4) There is no default limit for **Scala** collection
    5) I've rebased from master branch
    
    Here is a screen capture showing the new default limit: 
    
![image](https://cloud.githubusercontent.com/assets/1532977/8129319/6fe1f298-1108-11e5-9708-f7891ea52fa1.png)
    
    For code normalization, I would prefer we merge and close this pull request 
and open a new JIRA to prepare the port of ZeppelinContext class in **Scala** 
with appropriate unit tests.
    
    Baby steps.
    
    
     


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