-
Notifications
You must be signed in to change notification settings - Fork 439
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
fix: removed abandoned package from composer #5685
fix: removed abandoned package from composer #5685
Conversation
Thanks for catching this. Do you know where this package is used? That way we can make sure the new package works (in case test coverage has gaps) |
I think Avro records are used in PubSub. Can we verify the system tests are working when we replace this? |
Thanks for this @vishwarajanand. I tested this out and got the following error: We could use a version of the avro library that supports 7.4+ and skip this test if it is run on a system that is lower than that req. |
Hi reviewers, thanks for looking into this. Per my understanding, AVRO is used in BigQuery/PubSub codebase mostly as an identifier to denote the AVRO type records. There are a couple of tests who pass AVRO record type and parse the results to asset on values. Ex: Schema::Type. |
@dwsupplee , great suggestion. ❤️🔥 |
Wikimedia/avro
package is abandoned and users are getting deprecating notice oncomposer update
from root.Per the deprecation flag on packagist website, upgrading to
rg/avro-php
to a latest version that supportsPHP>=5.6
.Links to packages:
https://packagist.org/packages/wikimedia/avro
https://packagist.org/packages/rg/avro-php#1.8.0
Internal bug: b/218995638