You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Local benchmarks show proto.Clone taking a small amount of CPU time (~8% total time). It's probably not enough to cause a regression but it's not great that something we introduced is taking a noticeable CPU time as it would decrease the impact of other performance improvements.
Clone is using reflection under the hood, we should try to minize its usage.
The text was updated successfully, but these errors were encountered:
kruskall
changed the title
Investigate proto.Clone performance issue
Investigate proto.Clone performance impact
Jun 21, 2023
func (p *YourProto) CloneVT() *YourProto: this function behaves similarly to calling proto.Clone(p) on the message, except the cloning is performed by unrolled codegen without using reflection. If the receiver p is nil a typed nil is returned.
Local benchmarks show
proto.Clone
taking a small amount of CPU time (~8% total time). It's probably not enough to cause a regression but it's not great that something we introduced is taking a noticeable CPU time as it would decrease the impact of other performance improvements.Clone is using reflection under the hood, we should try to minize its usage.
The text was updated successfully, but these errors were encountered: