search

dbos-inc / dbos-transact

The TypeScript framework for backends that scale
https://docs.dbos.dev
MIT License
335 stars 22 forks source link

DRY out flushing result buffer #403

Closed devhawk closed 3 months ago

devhawk commented 4 months ago

There seems to be some duplicate logic in flushResultBuffer and flushWorkflowResultBuffer. Should we DRY this out?

kraftp commented 3 months ago

I took a close look at those methods--while there's some code that's similar, it's not exactly the same, and the differences are important. For now, trying to merge them will likely just make them more complex.