Massive console errors when changing biome with SQLite database

[mindw0rm]

Bug Report

Description

I'm running Bentobox and BSkyblock 1.8.0, along with the newest biomes snapshot (and a bunch of other addons).
After submitting the Biomes GUI, the server console is spammed with error messages of the following type:
[20:50:00] [Craft Scheduler Thread - 25/ERROR]: [BentoBox] Could not delete object world.bentobox.biomes.database.objects.BiomeChunkUpdateObject BSkyBlock_-15--1 Delete did not affect any rows!
[20:50:00] [Craft Scheduler Thread - 25/ERROR]: [BentoBox] Could not delete object world.bentobox.biomes.database.objects.BiomeChunkUpdateObject BSkyBlock_-14--1 Delete did not affect any rows!
They start with -15, -1 and finally (3.5 minutes later) end with 8,31 - presumably all chunks of my island.

Steps to reproduce the behavior:

1. enter /island biomes
2. Click on some biome
3. Click on island (well, it's already selected), and click accept
4. check server console

When setting for the whole island, there were the massive errors shown above.
On the first try (which was with an outdated spigot version (maybe 7-10 days old)), the server console was not responsive afterwards, shutting it down with Ctrl+C worked, but took quite some time. On the second try, the server was still responsive; in both tries there was no success output and no autput regarding the offline updater, and debug info in the clienst still showed the old biome - only after a relogin, or dying, or teleporting to another world (acid island) and returning the new biome was shown (but I think this might be a Minecraft issue).

I tried other settings as well:

• chunk with value=3 or value=5 gave no errors, a success message was shown both in the server console and in the client.
• the same with range and value=8 or value=16, or value = 32.
• chunk + value=8 gave some error messages of the same type as above, but also success messages. It also showed Populated offline updater with 226 chunks.. chunk + value=16 gave a lot of error messages as well, but also a success message, and only 223 chunks for the offline updater ...

The last message makes me assume that the problem are chunks that are not loaded.

Expected behavior

No errors in the console.

Screenshots or video

n/a

Server Information:

[Please complete the following information:]

• Database being used (YAML, JSON, MySQL, MongoDB): SQLite
• OS: Server Lubuntu 18.04, Client Windows 7
• Java Version: openjdk 1.8.0_222 (i.e. Java 8)
• Spigot version: git-Spigot-56f8471-7554e08 (newest version)
• BentoBox version: 1.8.0
• Addons installed?
  [21:11:43] [Server thread/INFO]: Running SPIGOT 1.14.4.
[21:11:43] [Server thread/INFO]: BentoBox version: 1.8.0
[21:11:43] [Server thread/INFO]: Database: SQLITE
[21:11:43] [Server thread/INFO]: Loaded Game Worlds:
[21:11:43] [Server thread/INFO]: acidisland_world (AcidIsland): Overworld, Nether, End
[21:11:43] [Server thread/INFO]: bskyblock_world (BSkyBlock): Overworld, Nether, End
[21:11:43] [Server thread/INFO]: caveblock-world (CaveBlock): Overworld, Nether, End
[21:11:43] [Server thread/INFO]: Loaded Addons:
[21:11:43] [Server thread/INFO]: AcidIsland 1.8.0 (ENABLED)
[21:11:43] [Server thread/INFO]: BentoBox-InvSwitcher 1.6.0 (ENABLED)
[21:11:43] [Server thread/INFO]: Biomes 1.6.0.2-SNAPSHOT-b142 (ENABLED)
[21:11:43] [Server thread/INFO]: BSkyBlock 1.8.0 (ENABLED)
[21:11:43] [Server thread/INFO]: CauldronWitchery 1.5.0.0-SNAPSHOT-#9 (ENABLED)
[21:11:43] [Server thread/INFO]: CaveBlock 1.8.0 (ENABLED)
[21:11:43] [Server thread/INFO]: Challenges 0.9.0-SNAPSHOT-#385 (ENABLED)
[21:11:43] [Server thread/INFO]: DimensionalTrees 1.6.0-SNAPSHOT (ENABLED)
[21:11:43] [Server thread/INFO]: Greenhouses 0.4.2-SNAPSHOT (ENABLED)
[21:11:43] [Server thread/INFO]: Level 1.8.0 (ENABLED)
[21:11:43] [Server thread/INFO]: Likes 1.7.0 (ENABLED)
[21:11:43] [Server thread/INFO]: MagicCobblestoneGenerator 1.7.0.2 (ENABLED)
[21:11:43] [Server thread/INFO]: Warps 1.8.0-SNAPSHOT-b187 (ENABLED)
• Other plugins? none

Additional context

View distance on the server is 10, on the client 20. 10 should be enough so that a chunk range of 8 should be completely within viewing distance, i.e. loaded. So maybe my assumption that only non-loaded blocks show errors is wrong?

[BONNe]

There is no any reason why you should use a snapshot version...
It is not the issue, but just to make it clear :) Release version is the best to use.

And it looks like it is some SQLite issue that prevents deleting objects.
@tastybento please can you look into it?

[mindw0rm]

Well I thought since the latest release is older than the release of bentobox 1.8.0, and only claims compatibility with 1.6.x, better try the snapshot ;)

But I can try again with the newest release.

[mindw0rm]

Would it be preferable to change to MySQL?
I saw no need to change the DB, so I kept the default.
However, I'm still evaluating (mainly which addons to use and how they work), so the server isn't open to everyone yet and starting from scratch would be acceptable.

Btw. Biomes 1.6.0.1 shows the same errors.

[mindw0rm]

I switched the database to MySQL (and reset the server), now everything works fine (with 3229 chunks for the offline updater! o.0 ).

So this is indeed a SQLite bug, and I guess it's not even a Biomes bug but a Bentobox bug - maybe you should move the ticket there?

[tastybento]

This is a bug in biomes where it calls a database delete on non-existent objects. The difference with MySQL and other databases is that a delete of a non-existent object fails silently. I'll upload a PR for @BONNe to look at.

[tastybento]

Fixed in BentoBox.