search

cmu-db / peloton

The Self-Driving Database Management System
http://pelotondb.io
Apache License 2.0
2.04k stars 623 forks source link

Sequences #1407

Open apavlo opened 6 years ago

apavlo commented 6 years ago

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

coveralls commented 6 years ago

Coverage Status

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

tli2 commented 6 years ago

@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.