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

Terminology updates, rename materialization(s) to table(s) and materialize() to publish() #58

Closed
4 tasks
lewish opened this issue Jan 11, 2019 · 0 comments
Closed
4 tasks
Assignees

Comments

@lewish
Copy link
Collaborator

lewish commented Jan 11, 2019

See terminology design doc here for complete explanation:

https://www.notion.so/dataform/Terminology-DD-d2ee07baee9548a686befa0e80069d04

  • Rename the global materialize(...) method to publish(...), but keep existing methods in place for backwards compatibility.
  • Rename CompiledGraph.materializations to CompiledGraph.tables
  • Rename Materialization proto and related classes to Table
  • Rename the existing Table protobuf to TableMetadata

Check for any other dangling references to the phrase "materialize" and update as appropriate.

@lewish lewish added this to the Beaver Public Beta milestone Jan 11, 2019
ShevtsovY added a commit that referenced this issue Jan 22, 2019
@lewish lewish closed this as completed Jan 23, 2019
@lewish lewish added queued and removed queued labels Jan 23, 2019
kolina pushed a commit that referenced this issue Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants