Document missing gRPC's default unit in GrpcClientProperties

[dlwldnjs1009]

GrpcClientProperties.Channel.Inbound.Message.maxSize and GrpcClientProperties.Channel.Inbound.Metadata.maxSize are missing the @DataSizeUnit(DataUnit.BYTES) annotation that their server-side equivalents in GrpcServerProperties have.

Server (annotated):

// GrpcServerProperties.Inbound.Message
@DataSizeUnit(DataUnit.BYTES)
private DataSize maxSize = DataSize.ofBytes(4194304);

// GrpcServerProperties.Inbound.Metadata
@DataSizeUnit(DataUnit.BYTES)
private DataSize maxSize = DataSize.ofBytes(8192);

Client (missing):

// GrpcClientProperties.Channel.Inbound.Message
private DataSize maxSize = DataSize.ofBytes(4194304);

// GrpcClientProperties.Channel.Inbound.Metadata
private DataSize maxSize = DataSize.ofBytes(8192);

The server-side Javadoc also documents "If a data size suffix is not specified, bytes will be used" while the client-side does not.

This is similar to gh-49722 where the @DurationUnit annotations were added consistently across gRPC properties. The commit 5b7a6a9 for that issue updated the Duration fields in GrpcClientProperties but the DataSize fields in the same class were not addressed.

[chhabil1605]

Hi, I’d like to work on this issue and submit a fix. Could you please confirm if it’s available for contribution?

[snicoll]

Thanks for the offer @chhabil1605 but I am looking at it already. I've assigned myself to make that more clear.

[snicoll]

Thanks @dlwldnjs1009 and sorry I overlooked those two!