Skip to content

Conversation

@wuyunfeng
Copy link
Member

@wuyunfeng wuyunfeng commented Jun 22, 2020

Add best practices for #3559 and update feature for #3901

@BabySid
Copy link
Contributor

BabySid commented Jun 22, 2020

lgtm

morningman
morningman previously approved these changes Jun 22, 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/docs Categorizes issue or PR as related to documentation. labels Jun 22, 2020
}
```

在Doris中建立该字段时建议设置为`date``datetime`,也可以设置为`varchar`类型, 使用如下SQL语句都可以直接将过滤条件下推至ES:
Copy link
Contributor

Choose a reason for hiding this comment

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

能否设置成char?

Copy link
Member Author

Choose a reason for hiding this comment

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

不建议用char,因为用户如果想那_id往往都是自己设置的,自己设置的ID长度大部分情况下可能并不一致

@wuyunfeng wuyunfeng requested a review from morningman June 22, 2020 15:07
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 merged commit e5da108 into apache:master Jun 23, 2020
@wuyunfeng wuyunfeng deleted the update-doe-doc branch June 23, 2020 05:43
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/docs Categorizes issue or PR as related to documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants