Skip to content

Fix H3 function name (h3ResIsClassIII to h3IsResClassIII)#29368

Merged
alexey-milovidov merged 5 commits intoClickHouse:masterfrom
bharatnc:ncb/fix-name-h3isResClassIII
Sep 26, 2021
Merged

Fix H3 function name (h3ResIsClassIII to h3IsResClassIII)#29368
alexey-milovidov merged 5 commits intoClickHouse:masterfrom
bharatnc:ncb/fix-name-h3isResClassIII

Conversation

@bharatnc
Copy link
Copy Markdown
Contributor

@bharatnc bharatnc commented Sep 25, 2021

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

Changelog category (leave one):

  • Not for changelog

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Rename H3 function h3ResIsClassIII to h3IsResClassIII to be consistent with the upstream H3 api.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Sep 25, 2021
@kitaisreal kitaisreal self-assigned this Sep 25, 2021
@bharatnc
Copy link
Copy Markdown
Contributor Author

@kitaisreal, my bad that I just noticed that this function name is slightly misplaced. It should be h3IsResClassIII instead of h3ResIsClassIII https://h3geo.org/docs/api/inspection#h3isresclassiii. I guess this is a bug fix (since it will be user visible). Also not sure about compatibility as I noticed that this function is now in v21.7.11.3-stable ?

@kitaisreal
Copy link
Copy Markdown
Contributor

@bharatnc from my perspective we should just merge it without backport or changelog.

@bharatnc
Copy link
Copy Markdown
Contributor Author

@bharatnc from my perspective we should just merge it without backport or changelog.

Sure, sounds good to me.

@kitaisreal kitaisreal removed the pr-bugfix Pull request with bugfix, not backported by default label Sep 25, 2021
@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Sep 25, 2021
@kitaisreal
Copy link
Copy Markdown
Contributor

@bharatnc could you please check tests failures ?

@bharatnc
Copy link
Copy Markdown
Contributor Author

@bharatnc could you please check tests failures ?

👍 missed renaming the reference file.

@robot-clickhouse robot-clickhouse added pr-not-for-changelog This PR should not be mentioned in the changelog and removed pr-bugfix Pull request with bugfix, not backported by default labels Sep 26, 2021
@alexey-milovidov alexey-milovidov mentioned this pull request Sep 26, 2021
@alexey-milovidov alexey-milovidov merged commit aa48698 into ClickHouse:master Sep 26, 2021
@alexey-milovidov alexey-milovidov self-assigned this Sep 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants