You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@alberginia Is there a reason for keeping the Queue managed outside of the SyntheticDataStreamer? I feel like this is an implementation detail that should be encapsulated inside the streamer class. The queue could be a member variable of the SyntheticDataStreamer class, set up automatically when the streamer instance is created. Then SyntheticDataStreamer could have a get() method which gets an item from the queue. The streamer instance would then be passed to the ContinuousCopy class rather than the queue. What do you think?
The text was updated successfully, but these errors were encountered:
@alberginia Is there a reason for keeping the Queue managed outside of the SyntheticDataStreamer? I feel like this is an implementation detail that should be encapsulated inside the streamer class. The queue could be a member variable of the SyntheticDataStreamer class, set up automatically when the streamer instance is created. Then SyntheticDataStreamer could have a
get()
method which gets an item from the queue. The streamer instance would then be passed to the ContinuousCopy class rather than the queue. What do you think?The text was updated successfully, but these errors were encountered: