Skip to content

Conversation

@goodboy
Copy link
Contributor

@goodboy goodboy commented Jul 8, 2022

Like it sounds, though turned out to be more involved then I would have liked in terms of getting our EMS messages moved over right away (at least with all the msgspec benefits).

See goodboy/tractor#311 for where we're probably going with all this.


TODO:

  • move over the piker.clearing._messages code
  • test thoroughly on multiple backends.
  • drop dep from install.

@goodboy goodboy requested a review from guilledk July 8, 2022 01:12
@goodboy goodboy marked this pull request as ready for review July 8, 2022 14:55
@goodboy goodboy requested a review from iamzoltan July 8, 2022 14:55
guilledk
guilledk previously approved these changes Jul 8, 2022
@goodboy goodboy changed the base branch from kraken_ws_orders to master July 9, 2022 17:08
@goodboy goodboy dismissed guilledk’s stale review July 9, 2022 17:08

The base branch was changed.

@goodboy goodboy requested a review from guilledk July 9, 2022 17:09
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