Skip to content

raise avro version to 1.12.1 and aggregate property usage#3334

Merged
derrickaw merged 5 commits intoGoogleCloudPlatform:mainfrom
derrickaw:20260214_avroVersionUpgrade
Feb 17, 2026
Merged

raise avro version to 1.12.1 and aggregate property usage#3334
derrickaw merged 5 commits intoGoogleCloudPlatform:mainfrom
derrickaw:20260214_avroVersionUpgrade

Conversation

@derrickaw
Copy link
Contributor

@derrickaw derrickaw commented Feb 14, 2026

fix #3173.

@gemini-code-assist
Copy link

Summary of Changes

Hello @derrickaw, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request centralizes and updates the Apache Avro dependency version across the project. By consolidating the Avro version definition in the root pom.xml and ensuring all submodules reference this single source, it aims to resolve potential version conflicts and streamline dependency management, specifically addressing an issue related to Avro versioning.

Highlights

  • Avro Version Update: The global Apache Avro dependency version has been updated from 1.11.4 to 1.12.1 in the main pom.xml.
  • Centralized Avro Version Management: Local Avro version properties in it/iceberg/pom.xml were removed, and the yaml/pom.xml now references the centralized ${avro.version} property, ensuring consistent versioning across modules.
Changelog
  • it/iceberg/pom.xml
    • Removed the local 'avro.version' property, allowing it to inherit from the parent POM.
  • pom.xml
    • Updated the 'avro.version' property from 1.11.4 to 1.12.1.
  • yaml/pom.xml
    • Changed the hardcoded Avro dependency version to use the '${avro.version}' property.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@derrickaw derrickaw added the improvement Making existing code better label Feb 14, 2026
@codecov
Copy link

codecov bot commented Feb 14, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.03%. Comparing base (18949a0) to head (a41fc2a).
⚠️ Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
.../teleport/v2/utils/DataplexJdbcPartitionUtils.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #3334      +/-   ##
============================================
- Coverage     51.04%   51.03%   -0.02%     
- Complexity     5213     5590     +377     
============================================
  Files           994      994              
  Lines         60672    60674       +2     
  Branches       6651     6652       +1     
============================================
- Hits          30972    30966       -6     
- Misses        27527    27533       +6     
- Partials       2173     2175       +2     
Components Coverage Δ
spanner-templates 70.86% <ø> (-0.02%) ⬇️
spanner-import-export 69.05% <ø> (-0.04%) ⬇️
spanner-live-forward-migration 79.83% <ø> (ø)
spanner-live-reverse-replication 77.46% <ø> (ø)
spanner-bulk-migration 87.93% <ø> (ø)
Files with missing lines Coverage Δ
.../teleport/v2/utils/DataplexJdbcPartitionUtils.java 57.62% <0.00%> (-2.03%) ⬇️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@derrickaw derrickaw marked this pull request as ready for review February 15, 2026 23:57
@derrickaw derrickaw changed the title raise avro version and aggregate property usage raise avro version to 1.12.1 and aggregate property usage Feb 16, 2026
@derrickaw derrickaw requested a review from Abacn February 17, 2026 13:58
Copy link
Contributor

@Abacn Abacn left a comment

Choose a reason for hiding this comment

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

Thanks!

@derrickaw derrickaw merged commit ab4b17c into GoogleCloudPlatform:main Feb 17, 2026
50 checks passed
@derrickaw derrickaw deleted the 20260214_avroVersionUpgrade branch February 17, 2026 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Making existing code better size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants