Skip to content

FieldMask: runtime + schema awareness#3644

Open
oldergod wants to merge 1 commit into
masterfrom
bquenaudon.2026-07-02.fieldmask_1
Open

FieldMask: runtime + schema awareness#3644
oldergod wants to merge 1 commit into
masterfrom
bquenaudon.2026-07-02.fieldmask_1

Conversation

@oldergod

@oldergod oldergod commented Jul 2, 2026

Copy link
Copy Markdown
Member

No description provided.

@oldergod oldergod marked this pull request as ready for review July 9, 2026 12:37
* Field masks are used to specify a subset of fields on a target message. Each path uses proto
* field names, separated by dots for nested fields.
*/
class FieldMask(paths: List<String> = emptyList()) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could this be a data class?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Any change are breaking for data classes, we have more control this way

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.

2 participants