-
Notifications
You must be signed in to change notification settings - Fork 27
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
Draft - Issue 989 big data support scaling vuu how to abstract vuu interfaces to allow different implementations of key classes by adding features on construction #1018
base: main
Are you sure you want to change the base?
Conversation
cleanup
cleanup
✅ Deploy Preview for papaya-valkyrie-395400 canceled.
|
feature/ignite-feature/pom.xml
Outdated
|
||
<dependency> | ||
<groupId>org.finos.vuu</groupId> | ||
<artifactId>order</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to have 'features' depend on particular 'use-cases' like 'order' or 'price' ?
I like the 'feature' and 'plugin' idea. This is very good 1st step towards making vuu a composable project |
I think this is good for understanding what it could potentially look like with the plugin concept & bootstraping relevant features etc. We may want to re-use the implementation in multiple plugin as well. E.g. VirtualDataTable for both Ignite and Hadoop.
I think there is still a lot of context im not aware of & normally find things i didnt think of at implementation time. Having a second implementation would help a lot in visualizing the right solution. Would it make sense as next step to have a spike of implementing the Virtual/BigData vuu solution using the ignite & once we have the second example, try and come up with the exact interfaces and how we can wire them up correctly? |
…ketch' into issue_989-big-data-support-scaling-vuu-how-to-abstract-vuu-interfaces-to-allow-different-implementations-of-key-classes-by-adding-features-on-construction
…nd all tests in vuu pass, but there is a problem with ignite module tests.
import org.finos.vuu.core.table.RowData | ||
import org.finos.vuu.table.virtualized.cache.GuavaRollingRowDataCache | ||
|
||
trait RollingCache[KEY, VALUE] { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wondering if 'RollingCache' is the most fortunate name for this trait ? The contract does not suggest anything 'rolling' here ? i.e. it can but does it have to ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, renamed from Rolling to Windowed
Good step forward, lets keep the ball rolling. |
…first sketch around what a fake big data provider might look like.
… be created based on in mem vs virtualized tables. This commit only contains the framework changes, next commit should contain the virtualized impl.
…ort-scaling-vuu-how-to-abstract-vuu-interfaces-to-allow-different-implementations-of-key-classes-by-adding-features-on-construction
…with querying ignite.
|
First example of a plugin structure for project.