isOpen

[dominictarr]

mentioned here:

levelgraph/levelgraph#53

it should just return whatever the root level returns for isOpen

[mcollina]

Thanks for opening this in here! I have been lazy with that and went for the short term fix, given that a rewrite of sublevel might happen #46 and the whole writestream removal business Level/levelup#207 is going to change everything anyway.

[dominictarr]

this would be a very easy change, and if levelup has it, sublevel should, just put it here to remind myself!

[elf-pavlik]

if levelup has it, sublevel should

how would you go about close()? @mcollina needed to wrap it while adding support for levelgraph on sublevel:
https://github.com/mcollina/levelgraph/blob/b133a18fc7ae8d660ab90b3bf41e301e682af8c8/lib/levelgraph.js#L77-L83

[dominictarr]

close should just deregister all event hooks, and take detach the sublevel object so that it can be GCd.

see discussion in this issue: #38

[TehShrike]

This came up in level-ttl a few months back. I think a good convention for levelup implementers would be to have a stop() function that removes all event hooks, with close() calling stop and also closing the underlying store.