Page MenuHomePhabricator
Differential D17090

Add Ngram support to Dashboards / Panels
ClosedPublic
Actions

Authored by chad on Dec 16 2016, 6:39 PM.
Tags
None
Referenced Files
F18821677: D17090.id.diff
Thu, Oct 23, 1:15 AM
F18783653: D17090.id41105.diff
Mon, Oct 13, 8:39 AM
F18641806: D17090.diff
Sep 18 2025, 9:48 PM
F18537670: D17090.id.diff
Sep 7 2025, 3:38 PM
F17875271: D17090.id.diff
Jul 28 2025, 6:14 PM
F17851094: D17090.diff
Jul 27 2025, 10:52 AM
F17848275: D17090.diff
Jul 27 2025, 7:11 AM
F17848204: D17090.id41105.diff
Jul 27 2025, 7:05 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rPc6bdd2c56bab: Add Ngram support to Dashboards / Panels
Summary

Build ngram indexs, adds search by name capability.

Test Plan

Search for a dashboard by partial name, search for a panel by partial name.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Dec 16 2016, 6:39 PM
Herald added a subscriber: Korvin. · View Herald TranscriptDec 16 2016, 6:39 PM
epriestley accepted this revision.Dec 16 2016, 8:06 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php
24

"badges" should be "dashboards".

This revision is now accepted and ready to land.Dec 16 2016, 8:06 PM
chad added inline comments.Dec 16 2016, 8:06 PM
src/applications/dashboard/query/PhabricatorDashboardPanelSearchEngine.php
54

this one's ok though? amirite?

chad added a comment.Dec 16 2016, 8:07 PM

daftpunk

Closed by commit rPc6bdd2c56bab: Add Ngram support to Dashboards / Panels (authored by chad, committed by chad). · Explain WhyDec 16 2016, 8:09 PM
This revision was automatically updated to reflect the committed changes.
epriestley added inline comments.Dec 16 2016, 8:09 PM
src/applications/dashboard/query/PhabricatorDashboardPanelSearchEngine.php
54

ACTUALLY IT LOOKS LIKE IT IS INDENTED TOO MUCH

frowncat

epriestley added a comment.Dec 16 2016, 8:13 PM

I am also shocked that you apparently encountered 0 fatals/exceptions/bugs while creating and landing this diff instead of the expected 50+.

Revision Contents
Changeset List

Diff 41107

View Options

resources/sql/autopatches/20161216.dashboard.ngram.01.sql

Loading...
View Options

resources/sql/autopatches/20161216.dashboard.ngram.02.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/dashboard/editor/PhabricatorDashboardPanelEditEngine.php

Loading...
View Options

src/applications/dashboard/query/PhabricatorDashboardPanelQuery.php

Loading...
View Options

src/applications/dashboard/query/PhabricatorDashboardPanelSearchEngine.php

Loading...
View Options

src/applications/dashboard/query/PhabricatorDashboardQuery.php

Loading...
View Options

src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php

Loading...
View Options

src/applications/dashboard/storage/PhabricatorDashboard.php

Loading...
View Options

src/applications/dashboard/storage/PhabricatorDashboardNgrams.php

Loading...
View Options

src/applications/dashboard/storage/PhabricatorDashboardPanel.php

Loading...
View Options

src/applications/dashboard/storage/PhabricatorDashboardPanelNgrams.php

Loading...
Phabricator · Built by Phacility