[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Hi @1ambda I created the #2421 --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 Hi, could you take care this issue might be related without NETWORK viz? - https://issues.apache.org/jira/browse/ZEPPELIN-2663 ```html ``` --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Thank you guys now I'll proceed with the second part of the original PR. The Neo4j's interpreter:) --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2125 Congrats! And thanks! --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 @felixcheung Yes, I already checked it but I was confused the branch name is master. :-) Thanks --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2125 looks like everything is green on travis but jenkins isn't picking them up https://travis-ci.org/conker84/zeppelin/builds/240773799 --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 Thanks, Will merge it after CI become green --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 Oh, I'm sorry for the late reply. I think you'd better to move those classes into another PR --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Hi guys, i'm still waiting an answer, maybe there is no interest to this PR, it's ok for me but just give me some feedback. Sorry if i bother you but I need this with the Neo4jIntepreter for a small PoC that i have to develop so I need infos in order to plan my work. Thanks --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Still no news? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 @jongyoul this contribution is the result of the split of #1582 in two little PR; the classes under `org.apache.zeppelin.interpreter.graph` provide the basic abstraction that can be used by graph dbs (like Neo4j, OrientDB, ArangoDB etc...) or graph processing frameworks (like GraphX or Giraph) and they will be used by the second part of the contribution which is the Neo4j interpreter, if you think is more appropriate I can move these classes into the second contribution. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 Hi, I have a question. I'm not sure but classes under org.apache.zeppelin.interpreter.graph is used only for testing it. Can you verify it? If it does, can you move these classes under test? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 No news? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Hi, could you please update me about the status of the review? Thanks! --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 It should be ok now --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 1. Regarding to `continuous-integration/appveyor/pr â AppVeyor build failed`, you should rebase. Since master has `.appvepor.yml` file. 2. In case of Jenkins, you are failing to profile 3 (selenium test) - [raw log](https://s3.amazonaws.com/archive.travis-ci.org/jobs/227364420/log.txt?X-Amz-Expires=30=20170504T121444Z=AWS4-HMAC-SHA256=AKIAJRYRXRSVGNKPKO5A/20170504/us-east-1/s3/aws4_request=host=1a7a934aff18da242587c547c957ace22807c6a84edcc70be96d48465c161611) ``` Tests in error: AuthenticationIT.testGroupPermission:190->AbstractZeppelinIT.pollingWait:94 û Timeout InterpreterIT.testShowDescriptionOnInterpreterCreate:69 û ElementNotVisible El... ``` They might be flaky tests. You can simply pass them by clicking `restart` button in travis. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 i tried but it still fails... --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 Hi, @conker84 Here is failed JOB and looks irrelevant. But could you make CI green? - https://travis-ci.org/conker84/zeppelin/jobs/227364420 you can click `restart` button in travis. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Is it now ok? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2125 Please rebase it upon upstream master branch --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Yes i made the rebase but it starts from the history **dc7b100** instead of **43352ae** and i can't understand why. Have you some ideas? Thanks --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2125 Do you mean the conflicts ? You need to do git rebase --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 I think i made some error with git commits please can someone help me? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Is it now ok? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 you should rebase based on master, not other branch. Currently, your change mixed with others.  --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Done! --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 FYI, i didn't mean you have to create other PRs. Others can improve UI. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Ok! So i'll rebase the code ASAP. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Let me know if you want go ahead with this PR or i have to close it! --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 UI can be improved in the following PRs. So it's ok not to resolve the whole things in this PR. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 We are talk about graphdb, the UI is an important part of the PR, as well as the new interpreter apis! I'm looking forward to your feedback. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 It's just my opinion. Thus, it's ok to use d3 if you think it's reasonable ð --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 Really sorry for late reply. I missed notification for a while. - Spell might use JSON output of neo4j interpreter utilizing [sigma.js](http://sigmajs.org/), [vis.js](http://visjs.org/network_examples.html) - Having default network display system using generalized graph model (Property Graph Model) would be good idea. - Personally, hope to get more refined, pretty graph UI instead of d3 network. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Yes it is! --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2125 I think this is introducing a new result type from interpreter on the backend side that could be useful for this new Neo4J interpreter and other graph-base ones in the future? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 @jongyoul thanks for the answer, i checked the [Spel](https://zeppelin.apache.org/docs/snapshot/development/writingzeppelinspell.html) and the [Visualization](https://zeppelin.apache.org/docs/latest/development/writingzeppelinvisualization.html) and i understood they are **front-end only**. This contribution is a mixed front/back-end becouse it want to provide a set of base api in order to enable graph-databases's exploration (like Neo4j or others) with Zeppelin. This contribution is the first step of #1582 the second is to implement the Neo4j Interpreter on top to the GraphResult apis. If you guys think there is a more valuable solution to this i'm happy to talk about! --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 Thanks for this nice contribution. We introduced Spell in the latest release. Spell helps you contribute this kind of wonderful visualization as easier than now. @1ambda Can you guide to convert it to Spell? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Hi guys i'm still here, let me know if i can do something to simplify the 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Hi guys any news/feedback? --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 I will test and give you feedback soon. --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user tae-jun commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 FYI, you can simply just take a GIF using software called **LICEcap**, and upload it directly here with drag and drop :) --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 @1ambda the graph is flatten by javascript apis so we can leverage the other visualizations provided by Zeppelin I'll add a video asap to show how it works! --- 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. ---
[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 Pretty graph! I have one question. - It's looks like new display system works with backend interpreter. - And enabled when we click the `network` visualization icon. So, what happens when we click 'table` or other visualization icons in the situation? (the screenshot you attached.) --- 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. ---
