Resource Control: Issues with the Version Referenced by Gosigar

This topic has been translated from a Chinese forum by GPT and might contain errors.

Original topic: resource control, gosigar 引用的版本问题

| username: ShawnYan

Why is cloudfoundry/gosigar referenced in TiDB, while elastic/gosigar is referenced in PD? Aren’t you worried that calling the same method from two different repositories might lead to synchronization issues and cause exceptions?

| username: redgame | Original post link

It might be due to historical reasons. Maybe one day it will be merged.

| username: GreenGuan | Original post link

Is this version 7.1?

| username: Anna | Original post link

Need an official response.

| username: ShawnYan | Original post link

Yes, the version 7.1 code of pd/tidb.

| username: HuSharp | Original post link

Thanks for the reminder, it was a mistake in importing the package.
However, this feature is not yet used in op, so the impact is not significant. I will submit a PR to make the changes as soon as possible.

| username: ShawnYan | Original post link


Related issue: resource_manage: fix gosigar import package · Issue #6734 · tikv/pd · GitHub
Already merged into master.

| username: system | Original post link

This topic was automatically closed 60 days after the last reply. New replies are no longer allowed.