Skip to content
This repository was archived by the owner on Jul 31, 2023. It is now read-only.

Move exporters out from core.#1118

Merged
songy23 merged 1 commit intocensus-instrumentation:devfrom
songy23:remove-exporters
Apr 23, 2019
Merged

Move exporters out from core.#1118
songy23 merged 1 commit intocensus-instrumentation:devfrom
songy23:remove-exporters

Conversation

@songy23
Copy link
Copy Markdown
Contributor

@songy23 songy23 commented Apr 19, 2019

Also run $ GO111MODULE=on go mod tidy.

@songy23 songy23 requested review from a team, rakyll and rghetia as code owners April 19, 2019 05:22
Copy link
Copy Markdown
Contributor

@rghetia rghetia left a comment

Choose a reason for hiding this comment

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

LGTM

@rghetia
Copy link
Copy Markdown
Contributor

rghetia commented Apr 19, 2019

after vanity-url PR is merged I can replace with new dependencies in examples
Ideally, it would be nice to create a separate repo for example so we don't haver circular dependencies.

@songy23
Copy link
Copy Markdown
Contributor Author

songy23 commented Apr 19, 2019

after vanity-url PR is merged I can replace with new dependencies in examples
Ideally, it would be nice to create a separate repo for example so we don't haver circular dependencies.

Agree, I'll hold off this PR until examples are updated. Or we can move the exporter-specific examples to exporter repos.

@rghetia
Copy link
Copy Markdown
Contributor

rghetia commented Apr 19, 2019

The problem with moving examples to exporter repo is that most of the example related to core functionality will move to prometheus just because we use prometheus in most (if not all) examples.

@bogdandrutu, should we create example repo and simply link it in readme.md in core library?

@bogdandrutu
Copy link
Copy Markdown
Contributor

Probably we can have in each exporter repo it's own example.

@rghetia
Copy link
Copy Markdown
Contributor

rghetia commented Apr 19, 2019

Probably we can have in each exporter repo it's own example.

That would require that we repeat feature examples (like gauge) in all exporters.

@songy23
Copy link
Copy Markdown
Contributor Author

songy23 commented Apr 23, 2019

Rebased against the latest dev branch (included #1128). Should be good to merge now.

@songy23 songy23 merged commit a10a38b into census-instrumentation:dev Apr 23, 2019
@songy23 songy23 deleted the remove-exporters branch April 23, 2019 20:26
rghetia pushed a commit to rghetia/opencensus-go that referenced this pull request Apr 25, 2019
rghetia pushed a commit that referenced this pull request Apr 25, 2019
@hsanjuan
Copy link
Copy Markdown

If anyone lands here wondering where they were moved to: https://github.com/census-ecosystem (with imports as contrib.go.opencensus.io/exporter/jaeger etc.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants