jupyter-xeus / xeus-sqlite

Jupyter kernel for SQLite
BSD 3-Clause "New" or "Revised" License
165 stars 25 forks source link

%CREATE instantly kills the kernel #109

Closed Epithumia closed 2 years ago

Epithumia commented 3 years ago

As the title says.

Tested on my install:

%CREATE "testdb.db"

results in "The kernel appears to have died. It will restart automatically." (and it doesn't, in fact). User log only shows the kernel restarting, but not what caused it to die.

Same thing happens with the example notebook ('Simple Operations').

Edit : once again, the documentation is wrong. %CREATE takes two arguments, but both doc and examples only pass one, which immediately kills the kernel.

In the example notebook, this command works fine:

%CREATE potato.db potato
marimeireles commented 3 years ago

Maybe it's the same problem with the "? Also, you're not giving a name to your DB in the first example, might be that. Anyways, it seems like we have a problem with outdated docs here...

Epithumia commented 3 years ago

It kills the kernel all the same without double quotes.

Anyway, there are two issues here:

marimeireles commented 3 years ago

Agreed, I thought I was handling it, but apparently not. Thanks for summarizing it clearly :) I can't take care of it right now, unfortunately, but it's good to keep track!

marimeireles commented 2 years ago

Fixed by https://github.com/jupyter-xeus/xeus-sqlite/pull/116