remove possibly redundant classes
the-james-burton opened this issue · 3 comments
Now that issue #16 is approaching completion, it is time to maybe refactor some of the original symbol/market handling. Possibly redundant classes include Stocks, Market and MarketsManager, which may no longer be necessary, since the Stock contains a Ticker which can be easily filtered in a java 8 stream.
This will affect the retrieval code in IndicatorSplitter and StrategySplitter. My initial guess is that MarketsManager will simply need a List and Stocks will disappear to be populated instead by initial data load from external source as detailed in issue #5 and maybe ultimately in ES via issue #2.
New TickerManager
class replaces the old classes. It is responsible for managing incoming Ticker
and Stock
objects and making them available to the rest of the turbine.
Reopening: StockRoute
and StockProcessor
believed redundant...
Indeed they are... removed.