Mark dagster.core private

[smackesey]

Summary & Motivation

Marks dagster.core private. Final step of incremental adoption of privacy for all submodules. Adds sys.meta_path mappings so that imports of e.g. dagster.core are mapped to dagster._core.

Changes were done by:

Using Pylance module rename refactor to perform automated module rename
Doing associated manual renames in the docs

Also includes some logic around ConfigurableClass:

• map dagster.X -> dagter._X in the ConfigurableClassData constructor.
• add a custom serializer to ConfigurableClassData that maps dagster._X to dagster.X

The thinking behind this was to (a) find the points where all old dagter.core references pass through and map the module name to the new name; (b) convert to the old name when serializing in case we are communicating with an older Dagster process.

Requires a sister PR in internal: https://github.com/dagster-io/internal/pull/2749

How I Tested These Changes

BK.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Ignored Deployments

Name	Status	Preview	Updated
dagit-storybook	⬜️ Ignored (Inspect)		Jul 23, 2022 at 11:30AM (UTC)
dagster	⬜️ Ignored (Inspect)		Jul 23, 2022 at 11:30AM (UTC)
dagster-oss-cloud-consolidated	⬜️ Ignored (Inspect)		Jul 23, 2022 at 11:30AM (UTC)

[smackesey]

Current dependencies on/for this PR:

• master
  • PR Remove submodule imports from doc snippets #8999
    • PR Public API types #8993
      • PR mark dagster.{api,cli,config} private #8829
        • PR Mark dagster.{daemon,generate,grpc,loggers} private #8884
          • PR Mark dagster.{builtins,experimental,serdes,seven,utils} private #8910
            • PR Fold primitive_mapping into dagster_type #8915
              • PR Remove requirement of ConfigurableClass from Dagster instance.yaml #8916
              • PR Mark dagster.core private #8981 👈
                • PR Typing improvements #9013

This comment was auto-generated by Graphite.