Henry LONNG

Results 60 comments of Henry LONNG

All you should do is that using protobuf protocol serializer before you call `starx.Run()` ```go starx.SetSerializer(protobuf.NewProtobufSerializer()) ``` In addition, the second parameter of your handler method should be a struct...

You can't combine JSON and Protobuf in the current version of StarX, but could combine for different server type. ```go starx.Set("server-type", func(){ starx.SetSerializer(json.NewJsonSerializer()) }) starx.Set("server-type-2", func() { starx.SetSerializer(protobuf.NewProtobufSerializer()) }) ```

Seems like you struct with wrong tags Try this: ```go type Message struct { Data string `protobuf:"bytes,1,name=data"` } ```

@zzzzzz7 Can you submit a pull request to fix it?

Oops, the result from cloud music server is empty.

I will address the conflicts after two reviewers LGTM.

> > @lonng @amyangfei The reason why engine change pkg name from 'pb' to 'enginepb' is : #5555 #5627 > > I think we need to do more work. >...

@amyangfei Why not change the `enginepb.WorkerInfo` to `pb.ExecutorInfo`, ``` message WorkerInfo { string id = 1; string executor_id = 2; bytes status = 3; bytes config = 4; bool is_tombstone...

This PR needs further work to merge and I will close it.