You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.
I now try to create a testfile (within the same path) e.g. for engine radix, with create_if_missing flag set up
I got error Failed creating pool: file .../testfile already exists
while I expect to get something like Failed opening pool: wrong layout ("..."), pool created with layout "..."
So, to sum up - using create_if_missing flag may conceal from the user the real problem with their pool.
Perhaps we should print both errors (one from the open method and the other one, from the create method) if applicable.
The text was updated successfully, but these errors were encountered:
Hm, yes, I think this might be a good idea. We would just need to make sure that in the documentation we have stated exactly what happens for create_if_missing (first create, then open) and also to clear the error if the second call succeeds.
Consider example:
create_if_missing
flag set upFailed creating pool: file .../testfile already exists
Failed opening pool: wrong layout ("..."), pool created with layout "..."
So, to sum up - using
create_if_missing
flag may conceal from the user the real problem with their pool.Perhaps we should print both errors (one from the
open
method and the other one, from thecreate
method) if applicable.The text was updated successfully, but these errors were encountered: