Skip to content

Conversation

@wuyunfeng
Copy link
Member

I forget push some UT test in this PR #4012, and remove _cluster/state resource because DOE does not rely the full ES cluster state meta.

@wuyunfeng wuyunfeng changed the title [Doris On ES] Add UT test for all search phase [Doris On ES] [UT] Add UT test for all search phase Jul 7, 2020
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morningman morningman added approved Indicates a PR has been approved by one committer. area/doris-on-es Issues or PRs related to Doris on ElasticSearch kind/test labels Jul 7, 2020
@morningman morningman merged commit 1cc9e16 into apache:master Jul 7, 2020
@wuyunfeng wuyunfeng deleted the fe-optimize branch July 8, 2020 01:34
morningman pushed a commit to morningman/doris that referenced this pull request Jul 9, 2020
I forget push some UT test in this PR apache#4012.
Also remove `_cluster/state` resource because DOE does not rely the full ES cluster state meta.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. area/doris-on-es Issues or PRs related to Doris on ElasticSearch kind/test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants