Skip to content

fix status don't update#125

Merged
tianxiaoliang merged 3 commits intoapache:masterfrom
GuoYL123:master
Mar 26, 2020
Merged

fix status don't update#125
tianxiaoliang merged 3 commits intoapache:masterfrom
GuoYL123:master

Conversation

@GuoYL123
Copy link
Contributor

No description provided.

@GuoYL123 GuoYL123 changed the title fix status/value_type don't update fix status don't update Mar 25, 2020
t.Run("list kv by service label, to create a polling data", func(t *testing.T) {
r, _ := http.NewRequest("GET", "/v1/test/kie/kv", nil)
noopH := &handler2.NoopAuthHandler{}
noopH2 := &handler2.TrackHandler{}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

noop 的意思是no operation

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

还是改下吧,1,2这种命名也是很奇怪的

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这是删掉的代码,新增的已经修改了

@tianxiaoliang tianxiaoliang merged commit feeacdf into apache:master Mar 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants