Skip to content

More processors for storage::read#8929

Merged
alexey-milovidov merged 70 commits intomasterfrom
more-processors-for-storages
Feb 21, 2020
Merged

More processors for storage::read#8929
alexey-milovidov merged 70 commits intomasterfrom
more-processors-for-storages

Conversation

@KochetovNicolai
Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Non-significant (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Processors pipeline for reading from other storages.

@compasses
Copy link
Copy Markdown
Contributor

Hello, cannot open the address: https://yandex.ru/legal/cla/?lang=en, which need yandex passport, so it's unavailable outside?

@alexey-milovidov
Copy link
Copy Markdown
Member

@compasses it's not required to sign the CLA. You can just read and copy the https://developercertificate.org/ instead.

@KochetovNicolai KochetovNicolai marked this pull request as ready for review February 20, 2020 14:53
pipeline.addSimpleTransform([&](const Block & source_header)
{
return std::make_shared<ConvertingTransform>(
source_header, header, ConvertingTransform::MatchColumnsMode::Name, context);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

FWIW (mostly for myself): fixes #9151

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.

4 participants