Remove misleading reference to "search_documentfield" in documentation
Changes PlannedPublic
Actions

Authored by epriestley on May 23 2019, 5:00 PM.

Details

Reviewers

amckinley

Maniphest Tasks

T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud)

Summary

Depends on D20549. Ref T11741. I removed this table in D20549, but both versions are slightly misleading because we actually use two MyISAM tables: "search_docmentfield" and "conpherence_index".

I just separated this out and would plan to land it after "conpherence_index" is removed so there are zero misleading lies at that point.

Test Plan

Grep / reading.

Diff Detail

Repository

rP Phabricator

Branch

myisam2

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 22899
Build 31412: Run Core Tests
Build 31411: arc lint + arc unit

Event Timeline

epriestley created this revision.May 23 2019, 5:00 PM

Harbormaster completed remote builds in B22899: Diff 49025.May 23 2019, 5:01 PM

epriestley requested review of this revision.May 23 2019, 5:01 PM

This looks good, except as you said, we can't land it until after we remove conpherence_index as well. Should I just approve it now and let it hang around in your unlanded revisions?

I'll just hold on to it for now in case something weird comes up, not a big deal if this doesn't land at the earliest possible moment.

Revision Contents
Changeset List

Path

Size

src/

docs/

contributor/

database.diviner

7 lines

Diff 49025

View Options

src/docs/contributor/database.diviner

Remove misleading reference to "search_documentfield" in documentationChanges PlannedPublicActions