Skip to content

improve graph operation performance (rebased from develop)#4400

Merged
joshmoore merged 1 commit intoome:metadatafrom
mtbc:adjust-graph-type-query-metadata
Feb 25, 2016
Merged

improve graph operation performance (rebased from develop)#4400
joshmoore merged 1 commit intoome:metadatafrom
mtbc:adjust-graph-type-query-metadata

Conversation

@mtbc
Copy link
Copy Markdown
Member

@mtbc mtbc commented Jan 6, 2016

With thanks to @joshmoore, adjust how graph traversal uses Hibernate to determine the exact class of model objects.

--rebased-from #4398 to metadata to assist testing

@mtbc mtbc changed the title improve graph operation performance improve graph operation performance (rebased from develop) Jan 6, 2016
@mtbc
Copy link
Copy Markdown
Member Author

mtbc commented Jan 6, 2016

Do we expect travis to pass on the metadata branch?

@jburel
Copy link
Copy Markdown
Member

jburel commented Jan 6, 2016

flake8 issue in the tests and web, that should be fixed so we have no problem when it is ported to the main line.

@joshmoore
Copy link
Copy Markdown
Member

Happy to have someone try, but if it were only flake8, I'm pretty sure I would have gone ahead and done it. If there isn't an immediate fix, it might be simpler to wait on the rebasing which will be ongoing throughout January.

@mtbc
Copy link
Copy Markdown
Member Author

mtbc commented Feb 24, 2016

Now #4398 is merged feel free to close or merge this as desired.

@joshmoore
Copy link
Copy Markdown
Member

I think I'll merge this to metadata, but won't rebase to metadata52 since it will be based on 5.2.2

joshmoore added a commit that referenced this pull request Feb 25, 2016
improve graph operation performance (rebased from develop)
@joshmoore joshmoore merged commit c256eba into ome:metadata Feb 25, 2016
@mtbc mtbc deleted the adjust-graph-type-query-metadata branch February 26, 2016 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants