Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

[Spark load][Be 1/1] Be handle push task #3742

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 11 commits into from
Jun 22, 2020
Merged

[Spark load][Be 1/1] Be handle push task #3742

merged 11 commits into from
Jun 22, 2020

Conversation

xy720
Copy link
Member

@xy720 xy720 commented Jun 1, 2020 •
edited
Loading

1、Add a PushBrokerReader in push_handle.cpp.
2、PushBrokerReader wraps the ParquetScanner to support reading data from parquet format file through broker.

#3433

@xy720 xy720 mentioned this pull request Jun 1, 2020
Closed
kangkaisen
kangkaisen reviewed Jun 1, 2020
View reviewed changes
be/src/olap/push_handler.h Outdated Show resolved Hide resolved
kangkaisen
kangkaisen reviewed Jun 1, 2020
View reviewed changes
be/src/olap/push_handler.cpp Outdated Show resolved Hide resolved
kangkaisen
kangkaisen reviewed Jun 1, 2020
View reviewed changes
be/src/olap/push_handler.cpp Show resolved Hide resolved
kangkaisen
kangkaisen reviewed Jun 1, 2020
View reviewed changes
be/src/olap/push_handler.cpp Outdated Show resolved Hide resolved
kangkaisen
kangkaisen reviewed Jun 1, 2020
View reviewed changes
be/src/olap/push_handler.cpp Show resolved Hide resolved
@xy720 xy720 force-pushed the spark_load_be_branch branch from 6dde2f9 to c97c3b0 Compare June 4, 2020 07:30
wyb
wyb reviewed Jun 4, 2020
View reviewed changes
be/src/olap/push_handler.cpp Outdated Show resolved Hide resolved
be/src/olap/push_handler.cpp Outdated Show resolved Hide resolved
be/src/olap/push_handler.cpp Show resolved Hide resolved
kangkaisen
kangkaisen reviewed Jun 19, 2020
View reviewed changes
be/src/olap/push_handler.cpp Outdated Show resolved Hide resolved
kangkaisen
kangkaisen reviewed Jun 19, 2020
View reviewed changes
be/src/olap/olap_common.h Outdated Show resolved Hide resolved
kangkaisen
kangkaisen previously approved these changes Jun 22, 2020
View reviewed changes
Copy link
Contributor

@kangkaisen kangkaisen left a comment

Choose a reason for hiding this comment

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

+1, LGTM

@xy720 xy720 requested a review from kangkaisen June 22, 2020 11:35
morningman
morningman approved these changes Jun 22, 2020
View reviewed changes
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 f189a2e into apache:master Jun 22, 2020
hubgeter pushed a commit to hubgeter/doris that referenced this pull request Mar 12, 2025
@bobhan1
[Fix](partial update) Fix incorrect result when partial update includ…
45e9719 
…e delete sign columns apache#46194 (apache#3742)

pick apache#46194
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wyb wyb wyb left review comments

@morningman morningman morningman approved these changes

@kangkaisen kangkaisen Awaiting requested review from kangkaisen

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@xy720 @wyb @morningman @kangkaisen
Morty Proxy This is a proxified and sanitized view of the page, visit original site.