Github user bzz commented on the issue:

    https://github.com/apache/zeppelin/pull/1371
  
    Thank you @Peilin-Yang ! 
    
    It looks like it's `O(N)` from the table size - do you think there might be 
a performance implication here? Just curious, but if it's a table - should not 
analysing just a few rows be enough?  
    
    @arunsoman that sounds interesting, but I think dev@ mailing list is much 
better place to discuss such changes, while keeping this PR to the scope of 
ZEPPELIN-1372.
    
    \cc @corneadoug for review


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to