Skip to content

filter applied to graph - #1414

Closed
lobamba wants to merge 1 commit into
apache:masterfrom
lobamba:graph_filter
Closed

filter applied to graph#1414
lobamba wants to merge 1 commit into
apache:masterfrom
lobamba:graph_filter

Conversation

@lobamba

Copy link
Copy Markdown

What is this PR for?

We were looking for a Business Intelligence tools that allow to make data analysis with nice charting.
We decided to use zeppelin that enables interactive data analytics.
We modified the source code to add more functionality:
For a first step we applied a filter to data.
User can now filter the data by clicking on graph. The filter is applied on every paragraph that have the same data(same column name).
On top you can see the filtered value.

What type of PR is it?

Improvement

What is the Jira issue?

  • No Jira created

How should this be tested?

graph filterd ### Questions: - Does the licenses files need update? No - Is there breaking changes for older versions? No - Does this needs documentation? No

@Leemoonsoo

Copy link
Copy Markdown
Member

Thanks @lobamba for the contribution.
It's interesting feature!

It'll be great if we can have more feedbacks and opinions here from the people, since this kind of feature has been never discussed in the community, as far as know. :-)

@placeybordeaux

Copy link
Copy Markdown

I'd personally like to see this PR land.

@pmccaffrey6

Copy link
Copy Markdown

This seems like a really cool idea and something that our group, for one, would love

@AhyoungRyu

Copy link
Copy Markdown
Contributor

@lobamba Great feature indeed!
As @Leemoonsoo said, I also would suggest to discuss about this newly proposed feature (maybe in dev@zeppelin.apache.org) first and get other ppl's opinion. It would be more helpful to proceed your work i think.

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lobamba@Leemoonsoo@placeybordeaux@pmccaffrey6@AhyoungRyu