[
https://issues.apache.org/jira/browse/METRON-1077?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16154334#comment-16154334
]
ASF GitHub Bot commented on METRON-1077:
----------------------------------------
Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/metron/pull/700#discussion_r137118292
--- Diff:
metron-interface/metron-alerts/src/app/service/cluster-metadata.service.ts ---
@@ -16,14 +16,20 @@
* limitations under the License.
*/
import {Injectable} from '@angular/core';
+import {Headers, RequestOptions} from '@angular/http';
import {Observable} from 'rxjs/Rx';
import {Http} from '@angular/http';
+
+import {INDEXES} from '../utils/constants';
+import {MetronRestApiUtils} from '../utils/metron-rest-api-utils';
import {ColumnMetadata} from '../model/column-metadata';
import {DataSource} from './data-source';
+import {HttpUtil} from '../utils/httpUtil';
@Injectable()
export class ClusterMetaDataService {
--- End diff --
Shouldn't this be in a search.service.ts class instead of having it's own
service class?
> Use column meta data end point in alerts ui
> -------------------------------------------
>
> Key: METRON-1077
> URL: https://issues.apache.org/jira/browse/METRON-1077
> Project: Metron
> Issue Type: Improvement
> Reporter: RaghuMitra
> Assignee: RaghuMitra
>
> Use the api in https://github.com/apache/metron/pull/662 PR instead of
> getting the fields from ES directly
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
