Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug with LMDB re-opening warnings #1017

Open
poodlewars opened this issue Oct 31, 2023 · 0 comments
Open

Bug with LMDB re-opening warnings #1017

poodlewars opened this issue Oct 31, 2023 · 0 comments
Assignees

Comments

@poodlewars
Copy link
Collaborator

poodlewars commented Oct 31, 2023

In [1]: from arcticdb import Arctic
ac = Arctic(f"lmdb:///tmp/one")

In [3]: lib = ac.get_library?

In [4]: lib = ac.get_library("test", create_if_missing=True)

In [5]: ac = Arctic(f"lmdb:///tmp/one")
[2023-10-31 11:09:58.710] [arcticdb] [warning] LMDB path at /tmp/one/ has already been opened in this process which is not supported by LMDB. You should only open a single Arctic instance over a given LMDB path. To continue safely, you should delete this Arctic instance and any others over the LMDB path in this process and then try again. Current process ID=[79057]

In [6]: lib = ac["test"]
[2023-10-31 11:10:09.197] [arcticdb] [warning] LMDB path at /tmp/one/test has already been opened in this process which is not supported by LMDB. To continue safely, you should delete all Library objects over the LMDB path in this process and then try again. This indicates a bug in ArcticDB. Please report at https://github.com/man-group/ArcticDB. Current process ID=[79057]

The second bit of logging here is misleading.

Also:

In [18]: ac.create_library("blah")
[2023-10-31 11:29:54.331] [arcticdb] [warning] LMDB path at /tmp/a/blah has already been opened in this process which is not supported by LMDB. To continue safely, you should delete all Library objects over the LMDB path in this process and then try again. This indicates a bug in ArcticDB. Please report at https://github.com/man-group/ArcticDB. Current process ID=[79755]
Out[18]: Library(Arctic(config=LMDB(path=/tmp/a)), path=blah, storage=lmdb_storage)
poodlewars added a commit that referenced this issue Oct 31, 2023
Otherwise, if a user ignores the first error, they are then spammed with misleading errors on their subsequent interactions.

See the Github issue for a couple of examples that this change seeks to avoid.
poodlewars added a commit that referenced this issue Oct 31, 2023
Otherwise, if a user ignores the first error, they are then spammed with misleading errors on their subsequent interactions.

See the Github issue for a couple of examples that this change seeks to avoid.
poodlewars added a commit that referenced this issue Oct 31, 2023
Otherwise, if a user ignores the first error, they are then spammed with misleading errors on their subsequent interactions.

See the Github issue for a couple of examples that this change seeks to avoid.
@poodlewars poodlewars self-assigned this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant