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

Add DatasetProvider interface. The clear advantage of this over simply renaming DatasetProviderBase to DatasetProvider is that current uses of DatasetProviderBase do not break (implementing DatasetProviders still extend DPBase). #431

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Feb 3, 2023

Add DatasetProvider interface. The clear advantage of this over simply renaming DatasetProviderBase to DatasetProvider is that current uses of DatasetProviderBase do not break (implementing DatasetProviders still extend DPBase).

@copybara-service copybara-service bot changed the title Rename DatasetProviderBase to DatasetProvider Add DatasetProvider interface. The clear advantage of this over simply renaming DatasetProviderBase to DatasetProvider is that current uses of DatasetProviderBase do not break (implementing DatasetProviders still extend DPBase). Mar 6, 2023
@copybara-service copybara-service bot force-pushed the test_479216470 branch 2 times, most recently from 0039107 to e184213 Compare March 16, 2023 00:05
…y renaming DatasetProviderBase to DatasetProvider is that current uses of DatasetProviderBase do not break (implementing DatasetProviders still extend DPBase).

PiperOrigin-RevId: 479216470
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

Successfully merging this pull request may close these issues.

None yet

1 participant