[
https://issues.apache.org/jira/browse/THRIFT-5420?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sylwester Lachiewicz reassigned THRIFT-5420:
--------------------------------------------
Assignee: Sylwester Lachiewicz
> Go library should not depend on "testing" in the main package
> -------------------------------------------------------------
>
> Key: THRIFT-5420
> URL: https://issues.apache.org/jira/browse/THRIFT-5420
> Project: Thrift
> Issue Type: Improvement
> Components: Go - Library
> Affects Versions: 0.14.0, 0.14.1
> Reporter: Yuri Shkuro
> Assignee: Sylwester Lachiewicz
> Priority: Minor
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> From [https://github.com/jaegertracing/jaeger-client-go/issues/585:]
> *Describe the bug*
> Importing {{jaeger-client-go}} adds a dependency on the standard library
> "testing" package. This is undesirable because production Go programs should
> not depend on the testing package. In addition to unnecessary bloat,
> "testing" adds a bunch of command line flags.
> The dependency is coming from
> [https://github.com/apache/thrift/blob/66d897667c451ef6560d89b979b7001c57a3eda6/lib/go/thrift/logger.go#L58]
> *Recommendation*
> Move TestLogger to a sub-module like "testutils". Or remove completely since
> it's a trivial implementation that can be easily reproduced. Or replace
> testing.TB argument with a locally defined interface that exposes Errorf
> function.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
