cloudwego / dynamicgo

Dynamically and efficiently operate RPC data for Go
Apache License 2.0
148 stars 28 forks source link

fix:(j2t) write redundant `base.Base` to thrift #55

Closed AsterDY closed 4 months ago

AsterDY commented 4 months ago

What type of PR is this?

Check the PR title.

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en: j2t.writeThriftBase has written base.Base to thrift message when enabling conv.Options.EnableThriftBase. But native.j2t_write_unset_fields and j2t.handleUnmatchedFields will still try to write this field when user not pass it in JSON (or pass null) zh(optional):

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation: