orientdb-studio icon indicating copy to clipboard operation
orientdb-studio copied to clipboard

Graph Editor - Query Execution - Add "Aggregate query results" checkbox

Open mccarvell opened this issue 11 years ago • 4 comments

I have two classes "Make" and "Model". In the graph view if I execute "select from Make" the correct nodes come into view.

If I then do "select from Model" I would expect that the "Make" nodes would be cleared out, but instead nodes from Make remain bringing up a two class view with both types of nodes.

mccarvell avatar Oct 14 '14 21:10 mccarvell

Yes this is done on purpose. The queries are appended in the Graph View in this way you are able to connect your Graph in a better way. If we clear the graph you will not able to connect Vertices. Think about

Person and Company

you want to connect Person to Company

you can do

select from Person where ...
select from Company where..

And then create edges in a graphical way.

wolf4ood avatar Oct 15 '14 10:10 wolf4ood

I see the use case, but it's inconsistent with the browse tab. There should be an option to invoke a different behaviour if it deviates. It makes sense to me to have the most consistent experience between browse and graph when it comes to executing API commands.

In your described use case above, I would do a Union (I think, still getting the hang of things!) to bring up nodes from both classes. Again, a checkbox for "append queries" could be considered as well. As an overall design, I think it makes sense to have a common history as well, so if I execute a query in browse, the history is there when I switch to graph. I think I sent you a mockup of this idea.

But seeing as it's not a bug, I guess we can categorize this as a suggestion.

mccarvell avatar Oct 15 '14 13:10 mccarvell

Yes we could tag this as enhancement and add the checkbox append queries I had your mockup but the image was only the browser tab and not the graph one

Could you send me also the graph mockup i will create a new issue with those improvements

Thanks for your feedback :D

wolf4ood avatar Oct 15 '14 13:10 wolf4ood

Hi @mccarvell, thanks for opening this issue

Current workaround: click on the "Clear Canvas" button before executing the second query

See also Issue #293. In that issue it was proposed to have "a graph representation" of the result set in another tab in the Browse UI ( Table / Raw - already existing - and Graph, the new tab).

We can also have, additionally, a checkbox "append queries" in the graph editor, enabled by default to preserve current behavior. When the "append queries" is false the graph editor would be cleared automatically

santo-it avatar Jan 02 '17 07:01 santo-it