#5086 Reindex errors on BlogPostSnapshot with no original()

v1.0.0
closed
sf-1 (616)
General
Cory Johns
2015-08-20
2012-10-08
No

If you delete a BlogPost, the BlogPostSnapshot entries stay. This makes them orphaned, and self.original() returns None, making things break: e.g. reindexing the project, historical search results presumably.

Discussion

  • Dave Brondsema

    Dave Brondsema - 2012-10-08
    • status: in-progress --> code-review
    • qa: Cory Johns
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-08

    allura:db/5086

    • delete a blog post, and its revisions should go away
    • if you have a blog post revisions from before this commit, running allurapaste reindex /var/local/config/production.ini -p myproj --solr --verbose shouldn't fail
    • note that BlogPostSnapshot revisions are not currently indexed by solr automatically. For some reason the BlogPostSnapshot object isn't in the ming session when the session closes at the end of the request. Didn't have time to dig into that
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-08
    • status: code-review --> open
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-08

    Ticket id issues also:

    4d18e4be1be1ce13e9000064
    4d18e4be1be1ce13e9000064
    4d38a4b2b9363c66830000a2
    4d38a4b2b9363c66830000a2
    4d38a4b2b9363c66830000a2
    4d973a830594ca4151000601
    4d973a830594ca4151000601
    4d973a830594ca4151000601
    4d973a830594ca4151000601
    4dc41bcc0594ca24e400007b
    4dc41bcc0594ca24e400007b
    4dc41bcc0594ca24e400007b
    4dc41bcc0594ca24e400007b
    4dc41bcc0594ca24e400007b
    4dc41bcc0594ca24e400007b
    4dc41bcc0594ca24e400007b
    4dc41bcc0594ca24e400007b
    
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-08

    Ticket nums are 1257, 1375, 2093, 4618 but we'll just leave them as is, the ticket's aren't too significant.

    • status: open --> code-review
     
  • Cory Johns - 2012-10-09
    • status: code-review --> validation
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-09
    • status: validation --> closed
     

Log in to post a comment.