Add Avro Configuration Properties to Hive connector docs#25311
Merged
beinan merged 1 commit intoprestodb:masterfrom Jun 18, 2025
Merged
Add Avro Configuration Properties to Hive connector docs#25311beinan merged 1 commit intoprestodb:masterfrom
beinan merged 1 commit intoprestodb:masterfrom
Conversation
martinsander00
requested review from
a team,
elharo and
steveburnett
as code owners
|
|
steveburnett
requested changes
Jun 13, 2025
Contributor
steveburnett
left a comment
steveburnett
left a comment
There was a problem hiding this comment.
Thanks for the documentation! Just a few suggestions. Looks good!
github-project-automation
bot
moved this from 🆕 Unprioritized
to 🏗 In progress
in Presto Documentation
Contributor
|
Now that you have documentation to link to, please add the doc link to the release note entry. Here's the formatting to use - I tested this in a local doc build and it worked. |
Contributor
|
Please sign the Presto CLA as mentioned in this comment. |
Contributor
|
@agrawalreetika and @imjalpreet, would you PTAL at this new doc when you can? |
imjalpreet
reviewed
Jun 13, 2025
imjalpreet
reviewed
Jun 13, 2025
martinsander00
force-pushed
the
fix-documentation-issue-24447
branch
2 times, most recently
from
7f40526 to
cfa7248
Compare
steveburnett
requested changes
Jun 17, 2025
Contributor
steveburnett
left a comment
steveburnett
left a comment
There was a problem hiding this comment.
Just a couple of formatting nits.
Introduce a new “Avro Configuration Properties” section describing: - and for Presto table DDL - for Hive 3.x Metastore (SerDeStorageSchemaReader) Include the XML snippet and note about restarting the metastore to avoid “Storage schema reading not supported” errors. Resolves: prestodb#24447
martinsander00
force-pushed
the
fix-documentation-issue-24447
branch
from
cfa7248 to
4873cf1
Compare
steveburnett
approved these changes
Jun 18, 2025
Contributor
steveburnett
left a comment
steveburnett
left a comment
There was a problem hiding this comment.
LGTM! (docs)
Pull updated branch, new local doc build, looks good. Thanks!
github-project-automation
bot
moved this from 🏗 In progress
to ✅ Done
in Presto Documentation
beinan
approved these changes
Jun 18, 2025
Member
beinan
left a comment
beinan
left a comment
There was a problem hiding this comment.
lgtm, thanks for the contribution!
6 tasks
12 tasks
This was referenced Jul 10, 2025
This was referenced Jul 24, 2025
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add Avro Configuration Properties to Hive connector docs
Introduce a new "Avro Configuration Properties" section describing:
hive.avro.schema.urlandhive.avro.schema.literalfor Presto table DDLhive.metastore.avro.schema.reader.enabledfor Hive 3.x Metastore (SerDeStorageSchemaReader)Include the XML snippet and note about restarting the metastore to avoid "Storage schema reading not supported" errors.
Resolves: #24447
Description
Adds documentation for Avro configuration properties:
hive.avro.schema.urlandhive.avro.schema.literalfor Presto table DDLhive.metastore.avro.schema.reader.enabledfor Hive 3.x Metastore (SerDeStorageSchemaReader)Include the XML snippet and note about restarting the metastore to avoid "Storage schema reading not supported" errors.
Motivation and Context
Adds Avro Configuration Properties so users with "java.lang.UnsupportedOperationException: Storage schema reading not supported" know how to address it.
Link to issue: #24447
Impact
None
Test Plan
None - documentation only change
Contributor checklist
Release Notes
Please follow release notes guidelines and fill in the release notes below.