Better error reporting on Pack In failure

Description

Tried to pack in some data and it failed with a NullPointerException. The stacktrace of the NPE helped me to establish that this was probably caused by a missing table, but it wasn’t very helpful in telling me which one.

I ended up putting a check in the code for the table lookup to see if the result is null and to throw an ISE with the table name if it fails. This allowed me to figure out which table was missing.

Thought that this little patch might be useful to have in HEAD. PR to follow.

Environment

None

Activity

Show:

Jeremy Krieg August 4, 2022 at 12:06 AM

PR is here:

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

Created August 4, 2022 at 12:01 AM
Updated October 1, 2022 at 7:25 AM
Resolved August 4, 2022 at 9:05 AM