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

[c++] Use std::future::get() to collect asynchronous query #2319

Closed
eddelbuettel opened this issue Mar 25, 2024 · 0 comments · Fixed by #2320
Closed

[c++] Use std::future::get() to collect asynchronous query #2319

eddelbuettel opened this issue Mar 25, 2024 · 0 comments · Fixed by #2320
Assignees

Comments

@eddelbuettel
Copy link
Contributor

eddelbuettel commented Mar 25, 2024

Describe the bug

It has been noticed that error conditions arising when queries are evaluated by the core engine did not properly propagate errors via expectations.

To Reproduce

Running a value filter against a column with an enumeration, and asking for a comparison with a non-existing value. This is covered by 2.21.0 and later, but errored with an exception up to 2.19.2. Running such a query from SOMA will demonstrate the issue

Versions (please complete the following information):

  • TileDB-SOMA version: 1.8.*
  • Language and language version (e.g. Python 3.8, R 4.2.2): any
  • OS (e.g. MacOS, Ubuntu Linux): any

Additional context

SC-43721

@eddelbuettel eddelbuettel self-assigned this Mar 25, 2024
@johnkerl johnkerl changed the title [c++] Use std::future::get() to collect asynchronous query [c++] Use std::future::get() to collect asynchronous query Mar 25, 2024
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

Successfully merging a pull request may close this issue.

1 participant