You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Yuxuan Wang (Jira)" <ji...@apache.org> on 2025/01/09 17:18:00 UTC

[jira] [Updated] (THRIFT-5845) The write error for union fields should be TException

     [ https://issues.apache.org/jira/browse/THRIFT-5845?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Yuxuan Wang updated THRIFT-5845:
--------------------------------
    Summary: The write error for union fields should be TException  (was: The write error for union fields should be of type of TTransportException)

> The write error for union fields should be TException
> -----------------------------------------------------
>
>                 Key: THRIFT-5845
>                 URL: https://issues.apache.org/jira/browse/THRIFT-5845
>             Project: Thrift
>          Issue Type: Task
>          Components: Go - Compiler
>    Affects Versions: 0.21.0
>            Reporter: Yuxuan Wang
>            Assignee: Yuxuan Wang
>            Priority: Major
>
> Currently, when writing an union field and not exactly one field of the union is set, we just return a generic error:
> {code:go}
> return fmt.Errorf("%T write union: exactly one field must be set (%d set)", p, c)
> {code}
> This makes it very hard for logic like client pool management to decide whether a connection can be reused after an error. Making it return a TTransportException would resolve the issue.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)