Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Elasticsearch scan reader #802

Merged
merged 17 commits into from
Mar 25, 2019

Conversation

wuyunfeng
Copy link
Member

No description provided.

std::string get_scroll_id();
bool count();
uint32_t total();
Status parse(std::string scroll_result);
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Status parse(std::string scroll_result);
Status parse(const std::string& scroll_result);


}

std::string ESScrollQueryBuilder::build_next_scroll_body(std::string scroll_id, std::string scroll) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
std::string ESScrollQueryBuilder::build_next_scroll_body(std::string scroll_id, std::string scroll) {
std::string ESScrollQueryBuilder::build_next_scroll_body(const std::string& scroll_id, const std::string& scroll) {

@lide-reed lide-reed self-requested a review March 25, 2019 07:52
Copy link
Contributor

@lide-reed lide-reed left a comment

Choose a reason for hiding this comment

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

LGTM

@lide-reed lide-reed merged commit 8c63b59 into apache:palo_on_es Mar 25, 2019
lide-reed pushed a commit that referenced this pull request Apr 8, 2019
lide-reed pushed a commit that referenced this pull request Apr 15, 2019
lide-reed pushed a commit that referenced this pull request Apr 18, 2019
lide-reed pushed a commit to lide-reed/incubator-doris that referenced this pull request Apr 18, 2019
lide-reed pushed a commit that referenced this pull request Apr 28, 2019
@wuyunfeng wuyunfeng deleted the feature-doris-on-es branch April 18, 2020 02:50
mrhhsg pushed a commit to mrhhsg/doris that referenced this pull request Oct 4, 2022
* topn detail query optimization: runtime predicate for rows

* add missing new files runtime_predicate.h/cpp

* topn detail query opt: runtime predicate for segments and pages

* topn detail query opt: fe check query to enable opt

* do not use topn opt for order by string type

* fix nullpointer when check type

* support more types and using TypeIndex in runtime_predicate

* [enhance-WIP](topn-two-phase) implement topn two phase read (apache#783)

* [fix](topn) fix conjunct_expr_root nullptr (apache#794)

* [enhance](topn-two-phase) support VOlapScanNode (apache#799)

* [enhance](topn) trick setParallelExecNum(1) when using topn optimization (apache#802)

* use HeapSorter when _use_topn_opt or no var length field

* remove debug log

Co-authored-by: Kang <kxiao.tiger@gmail.com>
liaoxin01 pushed a commit to liaoxin01/doris that referenced this pull request Nov 18, 2022
* Opt perf topn (apache#805)

* topn detail query optimization: runtime predicate for rows

* add missing new files runtime_predicate.h/cpp

* topn detail query opt: runtime predicate for segments and pages

* topn detail query opt: fe check query to enable opt

* do not use topn opt for order by string type

* fix nullpointer when check type

* support more types and using TypeIndex in runtime_predicate

* [enhance-WIP](topn-two-phase) implement topn two phase read (apache#783)

* [fix](topn) fix conjunct_expr_root nullptr (apache#794)

* [enhance](topn-two-phase) support VOlapScanNode (apache#799)

* [enhance](topn) trick setParallelExecNum(1) when using topn optimization (apache#802)

* use HeapSorter when _use_topn_opt or no var length field

* remove debug log

Co-authored-by: Kang <kxiao.tiger@gmail.com>

* [chore](topn-optimize) make code more readable

Co-authored-by: Kang <kxiao.tiger@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants