Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename reports to exposures #2777

Closed
jtcohen6 opened this issue Sep 21, 2020 · 1 comment
Closed

Rename reports to exposures #2777

jtcohen6 opened this issue Sep 21, 2020 · 1 comment
Labels
enhancement New feature or request
Milestone

Comments

@jtcohen6
Copy link
Contributor

Describe the feature

We've had several conversations internally since #2730. While concrete, report names only a subset of the intended functionality of the new node type. We're going back to the original name, exposure, because:

  • it accords with the intended superset of use cases
  • it better communicates to dbt developers the why of this feature

Describe alternatives you've considered

Other names we considered:

  • interfaces: good, but too vague/broad. This is a word we use often when training and talking about dbt, e.g. the way it interfaces with different databases.
  • contracts: good; related but slightly different. Each dbt model might have its own contract; that one model could be exposed in different ways, however, in different downstream dashboards/reports/notebooks/applications.

Additional context

All functionality is identical. The only proposed change is replacing all instances of report(s) with exposure(s).

Future functional changes to the exposures node, out of scope here:

  • Add support for tags, such that dbt ls -s tag:common_tag will return all sources, models, tests, and exposures that have common_tag

Who will this benefit?

Names are important, naming things is hard. We want to be using the best words we can when it's time to put this in front of dbt developers.

@jtcohen6 jtcohen6 added the enhancement New feature or request label Sep 21, 2020
@jtcohen6 jtcohen6 added this to the 0.18.1 milestone Sep 21, 2020
@beckjake beckjake mentioned this issue Sep 21, 2020
4 tasks
@beckjake
Copy link
Contributor

Fixed in core with #2780

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants