Uploaded image for project: 'Stanbol (Retired)'
  1. Stanbol (Retired)
  2. STANBOL-833

Feedbacks on SolrStoreImpl

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • contenthub-0.10.0
    • Contenthub
    • None

    Description

      Here come a patch for review, as I may miss some things.
      3 modifications here :

      1) Move the updateEnhancementGraph(ci); call from "enhance" to "put", as updateEnhancementGraph(ci); fit better in "put" as it's the clerezza graph counterpart of Solr save.
      So now, "enhance" only enhance and "put" save your CI (in the clerezza graph and in the Solr Core).

      2) Change a triple iteration to the build-in graph.addAll() function. There is a special reason to use an iteration here ?

      3) Remove code duplication in the "put(ContentItem ci)" and "put(ContentItem ci, String ldProgramName)"

      Patch attached for comments.
      Thanks.

      Attachments

        1. Stanbol-833.patch
          4 kB
          Florent ANDRE

        Activity

          People

            Unassigned Unassigned
            flo4sengines Florent ANDRE
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: