Fix: update glue datacatalog hive3 client jar npe - #361
Merged
jamespfaulkner merged 2 commits intoAug 3, 2026
Conversation
Rebuilds the bundled aws-glue-datacatalog-hive3-client-3.4.0-WD-1 jar from branch/waggle-dance with JDK 8 (matching the pom's 1.8 source/target), picking up upstream commit b52cbc9 which guards BaseCatalogToHiveConverter against a null SerdeInfo and null Compressed/NumberOfBuckets/StoredAsSubDirectories fields on Iceberg tables whose StorageDescriptor was stripped (e.g. by Glue's managed column-statistics service). The previous jar rebuild for this fix was built with JDK 25 by mistake; verified via javap bytecode diff against the pre-fix jar that the null-guards are present and the class files target Java 8. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Every recent CHANGELOG entry (including the immediately preceding 4.1.8 entry) is committed under a resolved [version] - date heading rather than [Unreleased]. pom.xml is already at 4.1.9-SNAPSHOT, so this entry should match that convention. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
jamespfaulkner
changed the title
jamespfaulkner
marked this pull request as ready for review
August 3, 2026 09:29
DhrubajyotiSadhu
approved these changes
Aug 3, 2026
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
2 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.
Summary
aws-glue-datacatalog-hive3-client-3.4.0-WD-1.jarwith JDK 8 (matching the fork's1.8source/target), picking up upstream fixb52cbc9which guardsBaseCatalogToHiveConverteragainst a nullSerdeInfoand nullCompressed/NumberOfBuckets/StoredAsSubDirectorieson Iceberg tables whoseStorageDescriptorwas stripped (e.g. by Glue's managed column-statistics service).Build-Jdk-Spec: 1.8) and class bytecode major version (52).Test plan
Build-Jdk-Spec: 1.8and class files are bytecode major version 52 (Java 8), matching WD's ownjdk.version=1.8compile target.Hive3CatalogToHiveConverterTest,GlueMetastoreClientDelegateTest,EntityConversionTest) against the rebuilt module — 140/140 pass, 0 failures.javap) the new jar'sBaseCatalogToHiveConverterandHiveTableValidatorclasses and diffed against the pre-fix jar (from before last week's Iceberg SD work) to confirm the null-guards (firstNonNullonCompressed/NumberOfBuckets/StoredAsSubDirectories, early-returnifnonnullcheck inconvertSerDeInfo) are actually present in the bundled bytecode, not just in source.🤖 Generated with Claude Code