Iterator returned from query ignores error handling

[Raynos]

The iterator from query() just returns key & `value.

const iterator = table.query(`hash = N(greetings) AND begins_with(range, S(hell))`)

for await (const { key, value } of iterator) {
  console.log(key, value)
}

Do we want

for await (const { err, data: { key, value } } of iterator) {

[heapwolf]

actually it throws. which is more conventional

[Raynos]

Same applies to scan()

[Raynos]

Throwing is fine, but then please also throw for table.get() and table.put() instead of returning { err }.

We should not mix the error handling of throw vs return { err } in the same codebase.

[heapwolf]

hmm. ok yeah.. { err, data } is more correct and more useful.

for await (const { err, data: { key, value } } of iterator) {