Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update github.com/parquet-go/parquet-go #18609

Merged
merged 1 commit into from
Sep 7, 2024
Merged

Conversation

reusee
Copy link
Contributor

@reusee reusee commented Sep 7, 2024

User description

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #18608

What this PR does / why we need it:

fix build


PR Type

enhancement


Description

  • Updated several dependencies in go.mod to their latest versions, including github.com/parquet-go/parquet-go, github.com/pierrec/lz4/v4, golang.org/x/sys, and google.golang.org/protobuf.
  • Corresponding updates to checksums in go.sum to reflect the new versions of the dependencies.

Changes walkthrough 📝

Relevant files
Dependencies
go.mod
Update dependencies in go.mod to latest versions                 

go.mod

  • Updated github.com/parquet-go/parquet-go from v0.20.1 to v0.23.0.
  • Updated github.com/pierrec/lz4/v4 from v4.1.18 to v4.1.21.
  • Updated golang.org/x/sys from v0.18.0 to v0.21.0.
  • Updated google.golang.org/protobuf from v1.33.0 to v1.34.2.
  • +8/-8     
    go.sum
    Update checksums in go.sum for new dependency versions     

    go.sum

  • Updated checksums for github.com/andybalholm/brotli to v1.1.0.
  • Updated checksums for github.com/klauspost/compress to v1.17.9.
  • Updated checksums for github.com/segmentio/encoding to v0.4.0.
  • Updated checksums for golang.org/x/sys to v0.21.0.
  • +16/-17 

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    Copy link

    qodo-merge-pro bot commented Sep 7, 2024

    PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Key issues to review

    Dependency Update
    Multiple dependencies have been updated to newer versions. Ensure these updates don't introduce breaking changes or incompatibilities.

    @matrix-meow matrix-meow added the size/S Denotes a PR that changes [10,99] lines label Sep 7, 2024
    @mergify mergify bot requested a review from sukki37 September 7, 2024 03:04
    Copy link

    qodo-merge-pro bot commented Sep 7, 2024

    PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Dependency update
    Update the parquet-go dependency to the latest version

    Update the github.com/parquet-go/parquet-go dependency to the latest version
    (v0.23.0) to ensure compatibility with other updated dependencies and potentially
    benefit from new features or bug fixes.

    go.mod [51]

    -github.com/parquet-go/parquet-go v0.20.1
    +github.com/parquet-go/parquet-go v0.23.0
     
    • Apply this suggestion
    Suggestion importance[1-10]: 9

    Why: The suggestion correctly identifies the update of the github.com/parquet-go/parquet-go dependency from v0.20.1 to v0.23.0, which is reflected in the PR code diff. This update is important for ensuring compatibility and potentially benefiting from new features or bug fixes.

    9
    Update the lz4 compression library dependency to the latest version

    Update the github.com/pierrec/lz4/v4 dependency to the latest version (v4.1.21) to
    ensure compatibility with other updated dependencies and potentially benefit from
    performance improvements or bug fixes in the LZ4 compression library.

    go.mod [52]

    -github.com/pierrec/lz4/v4 v4.1.18
    +github.com/pierrec/lz4/v4 v4.1.21
     
    • Apply this suggestion
    Suggestion importance[1-10]: 9

    Why: The suggestion accurately reflects the update of the github.com/pierrec/lz4/v4 dependency from v4.1.18 to v4.1.21, as shown in the PR code diff. This update is beneficial for compatibility and potential performance improvements.

    9
    Update the sys package dependency to the latest version

    Update the golang.org/x/sys dependency to version v0.21.0 to ensure compatibility
    with other updated dependencies and potentially benefit from the latest system-level
    functionality and bug fixes.

    go.mod [72]

    -golang.org/x/sys v0.18.0
    +golang.org/x/sys v0.21.0
     
    • Apply this suggestion
    Suggestion importance[1-10]: 9

    Why: The suggestion correctly identifies the update of the golang.org/x/sys dependency from v0.18.0 to v0.21.0, which matches the changes in the PR code diff. This update is crucial for compatibility and accessing the latest system-level functionality.

    9
    Update the protobuf dependency to the latest version

    Update the google.golang.org/protobuf dependency to version v1.34.2 to ensure
    compatibility with other updated dependencies and potentially benefit from the
    latest Protocol Buffers functionality and performance improvements.

    go.mod [75]

    -google.golang.org/protobuf v1.33.0
    +google.golang.org/protobuf v1.34.2
     
    • Apply this suggestion
    Suggestion importance[1-10]: 9

    Why: The suggestion accurately reflects the update of the google.golang.org/protobuf dependency from v1.33.0 to v1.34.2, as indicated in the PR code diff. This update is important for compatibility and leveraging the latest Protocol Buffers functionality and performance improvements.

    9

    @mergify mergify bot merged commit dc4d217 into matrixorigin:1.2-dev Sep 7, 2024
    17 of 18 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    4 participants