-
Notifications
You must be signed in to change notification settings - Fork 7
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 constructor that takes any IterableTable
or DataStream.Source
#23
Comments
I was thinking about this in EcoBase, is that repo mothballed for now? |
No not at all mothballed but EcoBase just contains abstract types and functions on them - no implementation. A constructor needs the concrete implementation of the type, thus I thought it should go here. |
Everything suffers a bit from me being a bit out of circulation as I'm writing a brand new 7-week MSc course starting in 13 days. |
Oh I see - that makes sense. And I didn't mean to impugn your efforts! I understand 💯% I love teaching, but it's a ton of work. |
This should use the new Tables interface. |
suggested by @kescobo
The text was updated successfully, but these errors were encountered: