Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-1153

deltaImportQuery should be honored on child entities as well

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4
    • Labels:
      None

      Description

      currently , only the root-entity can have this attribute

        Activity

        Hide
        archanak Archana Reddy added a comment -

        We are facing a similar issue but out use case is nested entity.

        we have the below nested entitties

        ParentEntity
        chilld1Entity
        child2Entity.

        child1Entity is child of ParentEntity and parent of child2Entity. Child2Entity is child of child1Entity.

        when we add deltaImportQuery on child2Entity. It is not executing deltaImportQuery during delta import.
        I found that below code(#503) in the DocBuilder class is restricting the deltaImportQuery from not being executed at the nested child entity level.
        for (EntityProcessorWrapper child : epw.getChildren())

        { buildDocument(vr, doc, child.getEntity().isDocRoot() ? pk : null, child, false, ctx, entitiesToDestroy); }
        Show
        archanak Archana Reddy added a comment - We are facing a similar issue but out use case is nested entity. we have the below nested entitties ParentEntity chilld1Entity child2Entity. child1Entity is child of ParentEntity and parent of child2Entity. Child2Entity is child of child1Entity. when we add deltaImportQuery on child2Entity. It is not executing deltaImportQuery during delta import. I found that below code(#503) in the DocBuilder class is restricting the deltaImportQuery from not being executed at the nested child entity level. for (EntityProcessorWrapper child : epw.getChildren()) { buildDocument(vr, doc, child.getEntity().isDocRoot() ? pk : null, child, false, ctx, entitiesToDestroy); }
        Hide
        gsingers Grant Ingersoll added a comment -

        Bulk close for Solr 1.4

        Show
        gsingers Grant Ingersoll added a comment - Bulk close for Solr 1.4
        Hide
        emtrane Martin Davidsson added a comment -

        Thank you, Noble. Does this mean any nightly build after 5/20 has your patch applied to it?

        Show
        emtrane Martin Davidsson added a comment - Thank you, Noble. Does this mean any nightly build after 5/20 has your patch applied to it?
        Hide
        noble.paul Noble Paul added a comment -

        committed revision: 776958

        Show
        noble.paul Noble Paul added a comment - committed revision: 776958
        Hide
        noble.paul Noble Paul added a comment -

        The fix ,I shall commmit this shortly

        Show
        noble.paul Noble Paul added a comment - The fix ,I shall commmit this shortly

          People

          • Assignee:
            noble.paul Noble Paul
            Reporter:
            noble.paul Noble Paul
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development