Page MenuHomePhabricator
Differential D21560

When documents are indexed, record the indexer version (versus the object version) and index epoch
ClosedPublic
Actions

Authored by epriestley on Feb 16 2021, 11:59 PM.
Tags
None
Referenced Files
F15464776: D21560.id51326.diff
Wed, Apr 2, 12:45 PM
F15456174: D21560.id51326.diff
Sun, Mar 30, 7:35 AM
F15452542: D21560.id51324.diff
Sat, Mar 29, 5:54 AM
F15448807: D21560.id51325.diff
Fri, Mar 28, 7:01 AM
F15442122: D21560.diff
Wed, Mar 26, 9:36 PM
F15441880: D21560.id51326.diff
Wed, Mar 26, 8:31 PM
F15418991: D21560.id51326.diff
Fri, Mar 21, 2:49 AM
F15285761: D21560.id51326.diff
Mar 4 2025, 3:56 PM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13587: Ferret may exhaust AUTO_INCREMENT ID space of "ngrams" table after many reindexes
Commits
rP6703fec3e27d: When documents are indexed, record the indexer version (versus the object…
Summary

Ref T13587. D21495 has significant changes to the ngram indexer, which might possibly contain bugs.

Make it easier to reindex a subset of documents (based on the date when the index was built, and/or the software version which generated the index).

This is in addition to the existing versioning, which is focused on object versions.

Test Plan

Ran bin/search index with various old and new arguments. Spot-checked the IndexVersion table.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 16 2021, 11:59 PM
Owners added a subscriber: Restricted Owners Package.Feb 16 2021, 11:59 PM
Harbormaster completed remote builds in B25134: Diff 51324.Feb 17 2021, 12:00 AM
epriestley requested review of this revision.Feb 17 2021, 12:00 AM
epriestley updated this revision to Diff 51325.Feb 17 2021, 12:03 AM
  • Fix typos and bugs, test more "--type" flag values.
Harbormaster completed remote builds in B25135: Diff 51325.Feb 17 2021, 12:04 AM
epriestley updated this revision to Diff 51326.Feb 17 2021, 12:06 AM
  • Correct "bin/index" -> "bin/search index".
Harbormaster completed remote builds in B25136: Diff 51326.Feb 17 2021, 12:07 AM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 17 2021, 12:09 AM
Closed by commit rP6703fec3e27d: When documents are indexed, record the indexer version (versus the object… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP6703fec3e27d: When documents are indexed, record the indexer version (versus the object….

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
2 lines
src/
applications/
search/
index/
29 lines
management/
323 lines
storage/
8 lines

Diff 51328

View Options

resources/sql/autopatches/20210216.index.01.version.sql

Loading...
View Options

resources/sql/autopatches/20210216.index.02.epoch.sql

Loading...
View Options

src/applications/search/index/PhabricatorIndexEngine.php

Loading...
View Options

src/applications/search/management/PhabricatorSearchManagementIndexWorkflow.php

Loading...
View Options

src/applications/search/storage/PhabricatorSearchIndexVersion.php

Loading...
Phabricator · Built by Phacility