Implement the spans to match the spec.

[schmidtw]

https://github.com/Comcast/wrp-c/wiki/Web-Routing-Protocol#simple-request-response-definition

Should be an array of structs not strings.

https://github.com/Comcast/wrp-c/blob/master/src/wrp-c.h#L67

[kristinapathak]

line of interest:
https://github.com/xmidt-org/wrp-go/blob/master/wrp/messages.go#L240

should be implemented to match this part of the spec:
(optional) An array of arrays of timing values as a list in the format: "parent" (string), "name" (string), "start time" (int), "duration" (int), "status" (int)

[iDevoid]

Hi @kristinaspring
Based on this doc I found that CRUD struct and Simple Request-Response has almost identical data structure.. CRUD has String path but Simple Request-Response has String span_parent..
I am sure both of the struct has different uses. but, any considering to merge these structs?

and based on this.. I found that CRUD also has Array.Array spans, but the go version of CRUD has Spans [][]string instead.. this is related, isn't this?

[kristinapathak]

Hi,

I believe @schmidtw has more information on the first point, but yes, I would think both CRUD and Simple Request-Response should have their spans value changed in the same way.

[schmidtw]

I found this PR in webpa-common that appears to do the work described here. Can we use this PR?

https://github.com/xmidt-org/webpa-common/pull/295/files