Skip to content
This repository was archived by the owner on Aug 15, 2024. It is now read-only.

Sqlite implementation #28

Closed
damianh opened this issue Jul 30, 2016 · 4 comments
Closed

Sqlite implementation #28

damianh opened this issue Jul 30, 2016 · 4 comments

Comments

@damianh
Copy link
Member

damianh commented Jul 30, 2016

No description provided.

@damianh
Copy link
Member Author

damianh commented Jul 30, 2016

Now that the SqlServer implementation is baking out a bit, time to do other storage implementations.

@dealproc
Copy link
Contributor

looking into it. going to have a lot of questions, but will see what i can accomplish. laying out project structures now, then will open the early pull request on this.

@dealproc
Copy link
Contributor

Functionally complete, but needs a massive refactor...
https://github.com/dealproc/SqlStreamStore/tree/sqlite-2

Thinking about two new methods (extension methods) off of DbCommand that would do something like:

var command = default(DbCommand);

// to get the $all stream
var all = command.AllStream();

// to work with a specific stream
var stream5 = command.streams("stream-5");
var meta5 = command.streams("$$stream-5");

There would be an underlying class for both the $all stream, and the per stream access. As an example:

public class AllStream {
    DbCommand _command;
    public AllStream(DbCommand command) {
        _command = command;
    }

    public long HeadPosition() {
        // sql statements to read head position from store.
        return (long)command.ExecuteScalar();
    }
}

My thinking here is that once the stream or the $all stream is captured, all commands/reads/etc can be performed and it'd all be within the command context.

@damianh
Copy link
Member Author

damianh commented May 18, 2020

I think we can close this :)

@damianh damianh closed this as completed May 18, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

2 participants