search

LumoSQL / archive1-LumoSQL-on-github

Initial work on SQLite and LMDB integration
https://lumosql.github.io/LumoSQL/
Other
68 stars 14 forks source link

DO NOT MERGE #13

Closed maxwell-k closed 4 years ago

maxwell-k commented 4 years ago

Need to quantify impact on speed

maxwell-k commented 4 years ago

Going to rebase so that:

  1. The change to btree.c which may impact performance is at the branch tip and
  2. To amend the path in the new copyright header to src/src/btree.c

In an ideal world I wouldn't put a directory called src inside another directory called src; I welcome suggestions of better names.

danshearer commented 4 years ago

Call it lmdb-backend?

Because this tree will only ever do one backend, lmdb. And its a nod to Howard. This tree will become deprecated relatively soon. Its our bootstrapper.

In a month or less I want to have a directory called say multi-backend. It'll be SQLite with multiple backends.

But then we will have something else which is our parser of choice with our backend of choice. And a lovely directory name we don't have to dream up yet.

Seem OK?

D

On Thu, 19 Dec 2019, 12:31 Keith Maxwell, notifications@github.com wrote:

Going to rebase so that:

  1. The change to btree.c which may impact performance is at the branch tip and
  2. To amend the path in the new copyright header to src/src/btree.c

In an ideal world I wouldn't put a directory called src inside another directory called src; I welcome suggestions of better names.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/LumoSQL/LumoSQL/pull/13?email_source=notifications&email_token=AA6DBAMQUCARL2JM65ZEHRLQZNSQJA5CNFSM4J4UCRE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHJOVZQ#issuecomment-567470822, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA6DBAJI6XLDVGV5IM6FTRTQZNSQJANCNFSM4J4UCREQ .

maxwell-k commented 4 years ago

Closting in favour of https://github.com/LumoSQL/LumoSQL/pull/14 https://github.com/LumoSQL/LumoSQL/pull/15 and https://github.com/LumoSQL/LumoSQL/pull/16