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.

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

Dataset Page: Metadata Display. Need More Efficient Method (solr, json, etc) #1822

Closed
raprasad opened this issue Mar 31, 2015 · 3 comments
Closed

Comments

@raprasad
Copy link
Contributor

Reconstruct Metadata values for display is costly in terms of queries that shoot to the top of the profiler*
(* 2nd after #1820)

Solr documents already contain metadata for display. Use these instead.

Longer terms: Save Metadata values as JSON (even if put into a Postgres field)

@raprasad raprasad added Priority: Medium Type: Suggestion an idea Component: Code Infrastructure formerly "Feature: Code Infrastructure" labels Mar 31, 2015
@raprasad raprasad added this to the 4.0.2 milestone Mar 31, 2015
@pdurbin
Copy link
Member

pdurbin commented Apr 1, 2015

Solr documents already contain metadata for display.

While this is true for the most part, the data you get out of Solr is flat key/value pairs. You lose all the hierarchy of compound fields.

@raprasad raprasad changed the title Dataset Page: Metadata Display. Use Solr Dataset Page: Metadata Display. Need More Efficient Method (solr, json, etc) Apr 1, 2015
@raprasad
Copy link
Contributor Author

raprasad commented Apr 1, 2015

(updated based on @pdurbin comments)

Short term strategy:

  • create table DatasetMetadataDisplay (or something similar)
dataset id dataset version id json metadata timestamp
  • For each version of metadata fields, look to the table above for info.
  • If not found:
    • Construct "display metadata" the current way
    • Save the constructed metadata as JSON to the table above, including citation block hierarchy.
    • Look at field ordering. Either put in JSON, or set field ordering dynamically...)
  • If found:
    • Update the DatasetPage.java / dataset.xhtml to use JSON as citation block metadata source

@scolapasta
Copy link
Contributor

No plan to do this for now. Can reopen, if/when we do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants