Page MenuHomePhabricator

Count and report skipped documents from "bin/search index"
ClosedPublic

Authored by epriestley on Apr 2 2017, 4:20 PM.
Tags
None
Referenced Files
F15535388: D17597.id.diff
Thu, Apr 24, 4:58 AM
F15532096: D17597.diff
Wed, Apr 23, 2:27 PM
F15504195: D17597.diff
Mon, Apr 14, 4:36 PM
F15498444: D17597.id42328.diff
Sun, Apr 13, 12:58 PM
F15467117: D17597.id42338.diff
Thu, Apr 3, 12:53 PM
F15455565: D17597.diff
Sun, Mar 30, 3:12 AM
F15449104: D17597.id42338.diff
Fri, Mar 28, 8:27 AM
F15385902: D17597.diff
Mar 14 2025, 11:38 PM
Subscribers
None

Details

Summary

Ref T12450. There's currently a bad behavior where inserting a document into one search service marks it as up to date everywhere.

This isn't nearly as obvious as it should be because bin/search index doesn't make it terribly clear when a document was skipped because the index version was already up to date.

When running bin/seach index without --force or --background, keep track of updated vs not-updated documents and print out some guidance. In other configurations, try to provide more help too.

Test Plan

Screen Shot 2017-04-02 at 9.17.23 AM.png (107×639 px, 30 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

It's definitely not obvious that objects are skipped based on the object version rather than what's in the index. So yeah this is helpful.

This revision is now accepted and ready to land.Apr 2 2017, 8:44 PM