peloton icon indicating copy to clipboard operation
peloton copied to clipboard

Published 20 hours ago • verified publisher icon cmu-db

Sequences

Open apavlo opened this issue 7 years ago • 2 comments

This PR is a cleaned up version of #1345. I removed the need for temp tables since @poojanilangekar doesn't need that. This PR is still broken. I still need to fix the basic functionality. I also debating whether to bring in the new SessionContext code from #1397.

https://www.youtube.com/watch?v=3mmdD0pBBck

apavlo avatar Jun 14 '18 14:06 apavlo

Coverage Status

Coverage decreased (-78.4%) to 0.0% when pulling 2d3fab887e3253c2932e72831a7f66876e55d94e on apavlo:sequences into 2676275c8ea298cb0fede43e2d9f838a0e6fb57e on cmu-db:master.

coveralls avatar Jun 14 '18 14:06 coveralls

@lmwnshn There seems to be a lot of refactors in this PR not necessarily related to sequences. We can probably get those into a separate PR and merge those immediately.

tli2 avatar Jun 26 '18 14:06 tli2