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

Main 202308091812 #13489

Merged
merged 42 commits into from
Aug 9, 2023
Merged

Main 202308091812 #13489

merged 42 commits into from
Aug 9, 2023

Conversation

ericsalo
Copy link
Member

@ericsalo ericsalo commented Aug 9, 2023

No description provided.

protobuf-team-bot and others added 30 commits July 11, 2023 14:54
MSVC compilers generate warnings that the default destructor is implicitly deleted, which causes errors on /W1. While this is a pedantic warning (error), we can easily make the deletion explicit to avoid these warnings.

PiperOrigin-RevId: 547548886
Explicitly delete the destructor for the internal `Rep` structure.
…google-cloud-shared-dependencies

PiperOrigin-RevId: 547882989
Put a 4k limit on preallocation to prevent OOM from malformed inputs.
PiperOrigin-RevId: 548128917
This will be used for tracking the unresolved feature sets from the original proto file.  Notable use-cases for this include:
* Code generators that need to validate their own features
* Runtimes that need to be able to accurately round-trip the original protos

PiperOrigin-RevId: 547610367
There are 4 different feature sets for every descriptor:
* Runtime/resolved - Used to make runtime decisions
* Source/resolved - Used to make codegen decisions
* Runtime/raw - Used to reason about the original proto file during runtime
* Source/raw - Used to validate features during codegen

All of these will need to be shipped to code generators.  While C++ generators can use the C++ runtime to reconstruct these, they won't end up in the FileDescriptorProtos we send to runtimes.  Similarly, non-C++ generators wouldn't be able to get them without duplicating our feature resolution logic.  This change adds a helper that allows us to bundle all of these into the protos we send to generators.

PiperOrigin-RevId: 547849243
There are 4 different feature sets for every descriptor:
* Runtime/resolved - Used to make runtime decisions
* Source/resolved - Used to make codegen decisions
* Runtime/unresolved - Used to reason about the original proto file during runtime
* Source/unresolved - Used to validate features during codegen

Plugins will receive all 4 of these.  Unresolved features will show up in the `raw_features` fields nested inside `features`.  The runtime features will be in the `proto_file` field of the request, while the source features will be in the unstripped `source_file_descriptors` field.

PiperOrigin-RevId: 547925275
Editions are still flag-guarded by the `--experimental_editions` flag for now, but once that's removed in a later release generators will need to explicitly specify that their support.  This will avoid cases where generators may happen to work for editions but produce incorrect code.

PiperOrigin-RevId: 547959326
Backport editions plugin improvements
…13296)

Prepare for 9.2 to be removed.

Closes #13296

COPYBARA_INTEGRATE_REVIEW=#13296 from protocolbuffers:jruby_version_bump 82924ab
PiperOrigin-RevId: 548279470
Add JRuby 9.4 tests and require Ruby version >=2.7
Update protobuf's upb dep to latest 24.x commit.
…eports of

compilers falling over due to the size of translation units.

PiperOrigin-RevId: 549653990
In OSS mode omit some extern template specializations. We have seen r…
This moves all our CI to Bazel 6 and drops the additional parameterization we had on Bazel version.

PiperOrigin-RevId: 551986376
@ericsalo ericsalo requested review from a team as code owners August 9, 2023 18:18
@ericsalo ericsalo requested review from JasonLunn, anandolee, googleberg, jskeet and fowles and removed request for a team August 9, 2023 18:18
@mkruskal-google mkruskal-google merged commit 3887b6f into main Aug 9, 2023
@mkruskal-google mkruskal-google deleted the main-202308091812 branch August 9, 2023 19:30
copybara-service bot pushed a commit that referenced this pull request Aug 9, 2023
Closes #13489

FUTURE_COPYBARA_INTEGRATE_REVIEW=#13489 from protocolbuffers:main-202308091812 e729f0b
PiperOrigin-RevId: 555243678
copybara-service bot pushed a commit that referenced this pull request Aug 9, 2023
Closes #13489

COPYBARA_INTEGRATE_REVIEW=#13489 from protocolbuffers:main-202308091812 e729f0b
FUTURE_COPYBARA_INTEGRATE_REVIEW=#13489 from protocolbuffers:main-202308091812 e729f0b
PiperOrigin-RevId: 555243678
copybara-service bot pushed a commit that referenced this pull request Aug 9, 2023
Closes #13489

COPYBARA_INTEGRATE_REVIEW=#13489 from protocolbuffers:main-202308091812 e729f0b
PiperOrigin-RevId: 555301642
protobuf-team-bot added a commit that referenced this pull request Aug 9, 2023
bithium pushed a commit to bithium/protobuf that referenced this pull request Sep 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants