Skip to content

chore: Use firestore-jar to determine grpc version#1076

Merged
lqiu96 merged 3 commits intomainfrom
main-fix_samples_deps
Oct 26, 2022
Merged

chore: Use firestore-jar to determine grpc version#1076
lqiu96 merged 3 commits intomainfrom
main-fix_samples_deps

Conversation

@lqiu96
Copy link
Copy Markdown
Member

@lqiu96 lqiu96 commented Oct 24, 2022

[WIP] This is to test our theory about the GRPC jars

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #1075 ☕️

@generated-files-bot
Copy link
Copy Markdown

Warning: This pull request is touching the following templated files:

  • samples/snapshot/pom.xml

@product-auto-label product-auto-label bot added the api: firestore Issues related to the googleapis/java-firestore API. label Oct 24, 2022
@lqiu96
Copy link
Copy Markdown
Member Author

lqiu96 commented Oct 24, 2022

It seems the beam dependencies were pulling in the older versions of the grpc-* jars (v1.48.0 when it needed v1.49.2). The firestore dependency gets updated with the new grpc-* versions via shared-deps and we should have the it pull in the dependency going forward.

I'll need figure out the warning with the snapshot jar being templated.

@lqiu96 lqiu96 added the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 26, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 26, 2022
@lqiu96 lqiu96 marked this pull request as ready for review October 26, 2022 15:50
@lqiu96 lqiu96 requested a review from a team October 26, 2022 15:50
@lqiu96 lqiu96 requested a review from a team as a code owner October 26, 2022 15:50
Copy link
Copy Markdown
Contributor

@dconeybe dconeybe left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for making this fix! The failing GitHub Action was such a nuisance. Plus, now the test coverage that the samples provide can be restored.

@lqiu96 lqiu96 merged commit dc7d220 into main Oct 26, 2022
@lqiu96 lqiu96 deleted the main-fix_samples_deps branch October 26, 2022 16:01
cherylEnkidu pushed a commit that referenced this pull request Dec 11, 2023
* chore: Use firestore-jar to determine grpc version

* chore: Add samples pom.xml to owlbot processor exlcusion list

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the googleapis/java-firestore API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

java.lang.NoClassDefFoundError: io/grpc/InternalMayRequireSpecificExecutor

2 participants