Skip to content

Improve geo parquet#80050

Merged
alexey-milovidov merged 4 commits intoClickHouse:masterfrom
scanhex12:impore_geo_parquets
May 11, 2025
Merged

Improve geo parquet#80050
alexey-milovidov merged 4 commits intoClickHouse:masterfrom
scanhex12:impore_geo_parquets

Conversation

@scanhex12
Copy link
Copy Markdown
Member

Support Nullable types in geo parquet and add setting to backward compatibility

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

...

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@alexey-milovidov alexey-milovidov added the can be tested Allows running workflows for external contributors label May 10, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented May 10, 2025

Workflow [PR], commit [92393cc]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 10, 2025
@alexey-milovidov alexey-milovidov self-assigned this May 11, 2025
@alexey-milovidov alexey-milovidov merged commit b884cb9 into ClickHouse:master May 11, 2025
115 of 120 checks passed
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label May 11, 2025
baibaichen pushed a commit to Kyligence/gluten that referenced this pull request May 15, 2025
baibaichen pushed a commit to apache/gluten that referenced this pull request May 15, 2025
* [GLUTEN-1632][CH]Daily Update Clickhouse Version (20250515)

* Fix build due to ClickHouse/ClickHouse#80035

* Fix build due to ClickHouse/ClickHouse#80050

---------

Co-authored-by: kyligence-git <gluten@kyligence.io>
Co-authored-by: Chang chen <chenchang@apache.com>
@alexey-milovidov
Copy link
Copy Markdown
Member

@scanhex12, still something is wrong with compatibility:

https://pastila.nl/?0098a044/b275f701ec9ed580074a4c775509c6c4#aTkwfEk61xPwxMVNxqKntA==

The types are inferred in a new way regardless of the compatibility flag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

can be tested Allows running workflows for external contributors pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants