Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

does not work with persistent level db #197

Open
scenaristeur opened this issue Jun 23, 2023 · 0 comments
Open

does not work with persistent level db #197

scenaristeur opened this issue Jun 23, 2023 · 0 comments

Comments

@scenaristeur
Copy link
Contributor

scenaristeur commented Jun 23, 2023 •
edited 

  a basic triple store
    ✔ persist, should put some more data inside a triple
    1) persist, should return the inserted data


  46 passing (123ms)
  1 failing

  1) a basic triple store
       persist, should return the inserted data:
     Uncaught TypeError: leveldb.createValueStream is not a function
      at Object.getStream (lib/levelgraph.js:59:28)
      at Object.get (lib/utilities.js:47:27)
      at /home/smag/dev/levelgraph/test/persist_properties_spec copy.js:27:10
      at /home/smag/dev/levelgraph/node_modules/abstract-level/abstract-level.js:589:7

https://github.com/Level/level/blob/master/UPGRADING.md#streams-have-moved
scenaristeur added a commit to scenaristeur/levelgraph that referenced this issue Jun 23, 2023
@scenaristeur
test for levelgraph#197
d1e221a
@scenaristeur scenaristeur mentioned this issue Jun 23, 2023
Open
scenaristeur added a commit to scenaristeur/levelgraph that referenced this issue Jun 26, 2023
@scenaristeur
create a persist-level test folder for levelgraph#197
76295b5
@scenaristeur scenaristeur mentioned this issue Jun 26, 2023
Open
mcollina pushed a commit that referenced this issue Apr 24, 2024
@finwo @scenaristeur
Update to use level 8 (#199)
e0163eb 
* level ^8.0.0 https://github.com/Level/level/blob/master/UPGRADING.md

* test for #197

* create a persist-level test folder for #197

* Update from memdb to level-mem ; remove implicit sublevel support

* Remove no-longer-supported approcimateSize during test inits

* Styling fixes to lib/readstream

* Ran browserify to update in-repo built files

* Using many-level instead of deprecated multilevel ; using level-read-stream instead of workaround

* jshint fixes

* Run browserify.sh to re-generate pre-built files

* Removed changelog.md, doesn't make sense in the context

* Update CI to only run on node 18/20

* Rewrote contributors table to gh-style markdown to familiarize myself with markdown again

* Removed broken dependency status link from readme

* Minor styling in readme on first few paragraphs

* Updated readme/install section to reflect current state

* Readme updates to reflect current initializers

* Removed unused level-iterator-stream dependency

* Update readme table of contents

* Nicer contributor alignment, like previous readme version

---------

Co-authored-by: Scenaristeur <scenaristeur@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@scenaristeur