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

DOCSUP-2965: Add translate for map functions #15744

Conversation

adevyatova
Copy link
Contributor

@adevyatova adevyatova commented Oct 7, 2020

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

Changelog category:

@robot-clickhouse robot-clickhouse added the pr-documentation Documentation PRs for the specific code PR label Oct 7, 2020
@adevyatova adevyatova changed the title Add translate for map func DOCSUP-2965: Add translate for map functions Oct 7, 2020
**Parameters**

Arguments are [tuples](../../sql-reference/data-types/tuple.md#tuplet1-t2) of two [arrays](../../sql-reference/data-types/array.md#data-type-array), where items in the first array represent keys, and the second array contains values for the each key.
All key arrays should have same type, and all value arrays should contain items which are promotable to the one type ([Int64](../../sql-reference/data-types/int-uint.md#int-ranges), [UInt64](../../sql-reference/data-types/int-uint.md#uint-ranges) or [Float64](../../sql-reference/data-types/float.md#float32-float64)).
The common promoted type is used as a type for the result array.
Copy link
Contributor

@BayoNet BayoNet Oct 14, 2020

Choose a reason for hiding this comment

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

If this sentence is supposed to be a separate paragraph, you must add the new line before it. Otherwise, it is not perceived as a separate paragraph.

What is it a common promoted type? Is it the resulting common type? Promotion is possibly the worst word that can be used in the meaning of type conversion.

Copy link
Contributor

Choose a reason for hiding this comment

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

Here and further.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Replaced with "reduced"

@BayoNet BayoNet merged commit ac438bc into ClickHouse:master Oct 15, 2020
@adevyatova adevyatova deleted the annadevyatova-DOCSUP-2965-mapPopulateSeries branch October 15, 2020 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-documentation Documentation PRs for the specific code PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants