Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Revert "Requst Inbound Relations" #875

Closed
jcoyne opened this issue Aug 13, 2015 · 0 comments
Closed

Revert "Requst Inbound Relations" #875

jcoyne opened this issue Aug 13, 2015 · 0 comments
Assignees
@jcoyne

Comments

@jcoyne
Copy link
Member

jcoyne commented Aug 13, 2015

This commit: 025a0fc

When getting inbound relations, the git request are from 3-5x slower than without. Furthermore the graphs are bigger, which means parsing takes longer.

This will probably break IndirectContainers (and maybe activefedora-aggregations).
#793

Ideally these requests could be turned on as needed.
@jcoyne jcoyne changed the title Revert RequstInboundRelations Revert "Requst Inbound Relations" Aug 13, 2015
@jcoyne jcoyne mentioned this issue Aug 13, 2015
Closed
jcoyne added a commit that referenced this issue Aug 14, 2015
@jcoyne
Stop using InboundRelationConnection
654fa7e 
The InboundRelation prefer header was causing a 3-5x performance
penalty. We can get the same info from Solr anyway.
Reverts #796
Fixes #875
@jcoyne jcoyne mentioned this issue Aug 14, 2015
Merged
@jcoyne jcoyne self-assigned this Aug 14, 2015
@jcoyne jcoyne self-assigned this Aug 14, 2015
jcoyne added a commit that referenced this issue Aug 18, 2015
@jcoyne
Stop using InboundRelationConnection
c8f08e0 
The InboundRelation prefer header was causing a 3-5x performance
penalty. We can get the same info from Solr anyway.
Reverts #796
Fixes #875
jcoyne added a commit that referenced this issue Aug 18, 2015
@jcoyne
Stop using InboundRelationConnection
fec345a 
The InboundRelation prefer header was causing a 3-5x performance
penalty. We can get the same info from Solr anyway.
Reverts #796
Fixes #875
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jcoyne jcoyne

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jcoyne @tpendragon