unable to import both major versions of appengine at the same time due to internal protos

[codyoss]

Please see issue for more context: googleapis/google-cloud-go#7760

Although theses deps should likely be updated it should not break users if they have both imported at the same time. The internal protos should have a new namespace for the v2 library to make the global protobuf registry happy.

[codyoss]

The issue:

appengine/internal/remote_api/remote_api.proto

Line 4 in f9c3e90

package remote_api;

[xStrom]

See also #281 & #228.

[codyoss]

Thanks for the cross links. If that issue is #281 is reopened I will close this. But I do believe this is a problem worth fixing or at least there needs to be more conversation on the issue to why it was closed. The Go module ecosystem is friendly to people importing multiple major versions at once -- having both import should not cause panics.

[xStrom]

I agree that it should be fixed. I'm stuck using old versions of unrelated libraries because of this issue.

[jinglundong]

having both import should not cause panics.

Yes, it's a fair behavior that we should support. We will reserve some time in our next sprint to at least working on a proposal.

[jinglundong]

After reading some of the related issues (googleapis/google-cloud-go#7760, #281 and #228), it appears that migrating from github.com/golang/protobuf to google.golang.org/protobuf is the short-term solution.

One common challenge of this type of issues is that Google's internal code base and OSS repos may diverge. We might be lucky here because this Github repo is the source of truth if I recall correctly. I can (or ask someone in our team) to give this change a shot.

[trakhimenok]

@jinglundong any estimates for resolution?

[jinglundong]

Sorry for being irresponsive on this issue. I will provide an estimate next week.

[jinglundong]

This issue aligns with an internal goal of our team. The current ETA is June 30th 2023.

[jinglundong]

An update: this issue didn't make to our current sprint ending on June 27th 2023. The new ETA is by the end of Q3 2023.

[xStrom]

Unfortunate of course, however I would like to thank you for keeping us informed. Very rare trait for a googler!