search

debasishg / frdomain

Code repo for Functional and Reactive Domain Modeling
Apache License 2.0
467 stars 136 forks source link

Minor Cleanups #16

Closed guersam closed 9 years ago

guersam commented 9 years ago

One last big rename (yes, we are truly sorry about that): while the semantics had evolved around it, we never adapted the name of the FlowMaterializer—which actually is not for Flows but all kinds of Graphs; similarly OperationAttributes have no matching “Operation” that they apply to. Hence we shortened and simplified

  • FlowMaterializer to Materializer
  • ActorFlowMaterializer to ActorMaterializer
  • OperationAttributes to Attributes
  • ActorOperationAttributes to ActorAttributes
  • RunnableFlow to RunnableGraph (for consistency, since this is not a Flow)
debasishg commented 9 years ago

Thanks a lot :-) .. I just did the same changes in the branch wip-ch7. One more refactoring that proved useful for RC4 is the helper function that Framing offers for parsing stages. See https://github.com/debasishg/frdomain/blob/wip-ch7/src/main/scala/frdomain/ch7/streams/TransactionProcessor.scala#L25 ..

guersam commented 9 years ago

Oh, I like it, much better than parseLines. Thanks for the pointer!