Github user 1ambda commented on the issue:

    https://github.com/apache/zeppelin/pull/2098
  
    @AhyoungRyu 
    
    Thanks for detailed review. 
    
    1. It would be nice to have some popup or feedback when conf is updated.
    2. Let me hide them when panel is minimized (e.g the chart panel, the 
parameter panel)
    
    Regarding to `3. no-group field`, it's quite hard to say in just few lines. 
So let me describe more.
    
    (A) Some charts require aggregation. (= groupBy in SQL)
    (B) But some charts might not need aggregation. Assume that it's already 
grouped by SQL
    
    In ultimate-line-chart, there are sub charts called `line`, `dashed`, 
`step` and `no-group`.
    
    - `line`, `dashed`,` step` uses aggregation like zeppelin's existing 
charts. (similar to `PivotTransformation`)
    - But `no-group` doesn't use aggregation. (similar to 
`ColumnTransformation`)
    



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