Skip to content
This repository has been archived by the owner on Nov 21, 2020. It is now read-only.

Upload files #1

Merged
merged 1 commit into from
Aug 20, 2016
Merged

Upload files #1

merged 1 commit into from
Aug 20, 2016

Conversation

jpiechowka
Copy link
Collaborator

No description provided.

@jpiechowka jpiechowka merged commit 9b4f5fa into openzipkin-attic:master Aug 20, 2016
});

const tracer = new Tracer({ctxImpl, recorder});
const nameOfRemoteService = 'cujojs-client';

Choose a reason for hiding this comment

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

Are you sure this is correct?

@jpiechowka
Copy link
Collaborator Author

jpiechowka commented Aug 20, 2016

@eirslett
This is what I used:
Zipkin-js-example
Zipkin-instrumentation-cujojs-rest

It works for me, the only issue with this right now is this, so I used dirty workaround of adding timeout:
Zipkin-js/issues/33

@eirslett
Copy link

I was unsure about the part "const nameOfRemoteService = 'cujojs-client';". I would suppose cujojs-client is the name of the local service, not the remote one?

@jpiechowka
Copy link
Collaborator Author

@eirslett Oh, yes it is name of the local service - the client. There is a screenshot in the repo if you want to see how it looks in Zipkin.

@eirslett
Copy link

Yes, that's why I thought maybe a name like "const nameOfClient = 'cujojs-client';" would be more descriptive?

@codefromthecrypt
Copy link

+1

On 21 Aug 2016 1:36 am, "Eirik Sletteberg" notifications@github.com wrote:

Yes, that's why I thought maybe a name like "const nameOfClient =
'cujojs-client';" would be more descriptive?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#1 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD61zI1N7qKOs3bF_jCqx7ILnYeCVubks5qhzs5gaJpZM4JpD7x
.

@jpiechowka
Copy link
Collaborator Author

@eirslett I have changed nameOfRemoteService to nameOfClient.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants