Page MenuHomePhabricator

elasticsearch host quick fix
ClosedPublic

Authored by fabe on May 21 2015, 3:44 PM.
Tags
None
Referenced Files
F18099685: D12967.id31238.diff
Sat, Aug 9, 7:40 AM
F18098004: D12967.id31237.diff
Fri, Aug 8, 5:43 PM
F18091542: D12967.id31239.diff
Wed, Aug 6, 10:33 PM
F18078351: D12967.id.diff
Mon, Aug 4, 8:04 PM
F18050374: D12967.diff
Sun, Aug 3, 7:30 PM
Unknown Object (File)
Jun 17 2025, 11:45 AM
Unknown Object (File)
Jun 17 2025, 8:42 AM
Unknown Object (File)
Jun 17 2025, 8:42 AM
Subscribers

Details

Summary

Quick fix for the elasticsearch search engine Ref T8274

Test Plan

Did a couple of searches

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

fabe retitled this revision from to elasticsearch host quick fix.
fabe updated this object.
fabe edited the test plan for this revision. (Show Details)
fabe added a reviewer: epriestley.

Can we just implement __construct().

That is:

public function __construct() {
  $this->uri = PhabricatorEnv::getEnvConfig('search.elastic.host');
  $this->index = PhabricatorEnv::getEnvConfig('search.elastic.namespace');
}
fabe edited edge metadata.

just use a constructor

just wasn't sure if there was a bette place to actually call setURI/Index.

epriestley edited edge metadata.

I don't think there is, currently.

This revision is now accepted and ready to land.May 21 2015, 4:01 PM
This revision was automatically updated to reflect the committed changes.