RumbleDB / rumble

⛈️ RumbleDB 1.22.0 "Pyrenean oak" 🌳 for Apache Spark | Run queries on your large-scale, messy JSON-like data (JSON, text, CSV, Parquet, ROOT, AVRO, SVM...) | No install required (just a jar to download) | Declarative Machine Learning and more
http://rumbledb.org/
Other
213 stars 82 forks source link

Error when accessing index 0 is confusing #1257

Open necessarily-equal opened 2 months ago

necessarily-equal commented 2 months ago

In cases where one tries to access the index of a sequence, no nice error message is printed but the frontend “panics” and prints a generic error message instead:

image

It could print a nice error message such as “tried to access index 0 of a sequence, in JSONiq indexes start at 1.” This may affect arrays also, I didn't test that case.

ghislainfourny commented 1 month ago

Thanks @necessarily-equal for this feedback. You are correct that this should not fail with such an error. We will address this.

mschoeb commented 1 week ago

Thanks for the report, turns out it was not just a bad error message but also a bug underneath. The fix for it is now commited in our non-public repo and will be merged sometime in the future