Skip to content

Data Initializer for sampleapp#86

Merged
srfc merged 20 commits intomasterfrom
feature/dataInitializer
Jun 14, 2018
Merged

Data Initializer for sampleapp#86
srfc merged 20 commits intomasterfrom
feature/dataInitializer

Conversation

@SebastianSchmidl
Copy link
Copy Markdown
Owner

@SebastianSchmidl SebastianSchmidl commented Jun 11, 2018

Fixes #53

Proposed Changes

  • Mixin for Dactors (called DataInitializer) handling loading data from CSVs into the internal relations
  • Mixin DefaultMessageHandling instead of overwriting unhandled in Dactor
  • Dactor responsible for system startup and loading our testsdata --> SystemInitializer
  • Manifest data in resources folder
  • let all sampleapp dactors support data initializing and default message handling
  • addition to CSVParser:
    • ability to process java.io.Streams
    • use these streams in DataInitializer
    • getClass.getResourceAsStream(name): InputStream does also work for resources inside jars, whereas getClass.getResource(name): File does not!

Related

@SebastianSchmidl SebastianSchmidl added the WIP Work In Progress label Jun 11, 2018
@SebastianSchmidl SebastianSchmidl self-assigned this Jun 11, 2018
@SebastianSchmidl SebastianSchmidl added Needs Review and removed WIP Work In Progress labels Jun 13, 2018
@SebastianSchmidl SebastianSchmidl requested a review from srfc June 13, 2018 16:31
@srfc srfc merged commit 633d0a8 into master Jun 14, 2018
@srfc srfc deleted the feature/dataInitializer branch June 14, 2018 08:32
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.

2 participants