Skip to content
New issue

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

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

Already on GitHub? # to your account

Improve Fetch-From-Type Warning in Log #1940

Open
gramian opened this issue Jan 31, 2025 · 3 comments
Open

Improve Fetch-From-Type Warning in Log #1940

gramian opened this issue Jan 31, 2025 · 3 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@gramian
Copy link
Collaborator

gramian commented Jan 31, 2025

ArcadeDB Version 25.1.1

Currently a repeated fetch from a non-indexed type causes a log entry like:

Attempt to scan type '%s' of total size %s %d times. This operation is very expensive, consider using an index

see: https://github.com/ArcadeData/arcadedb/blob/main/engine/src/main/java/com/arcadedb/query/sql/executor/FetchFromTypeExecutionStep.java#L117 .
This is a very useful help for a database designer, but it could still be improved by adding:

  • the to-be-fetched property or bucket name. (This makes only sense if a bucket name is available and not only a number)
  • the name of the database in which the to-be-fetched type is.
@gramian gramian added the enhancement New feature or request label Jan 31, 2025
@robfrank robfrank added this to the 25.2.1 milestone Jan 31, 2025
@robfrank robfrank self-assigned this Jan 31, 2025
@lvca
Copy link
Contributor

lvca commented Feb 2, 2025

What do you mean with "the to-be-fetched property or bucket name. (This makes only sense if a bucket name is available and not only a number)"?

lvca added a commit that referenced this issue Feb 2, 2025
@gramian
Copy link
Collaborator Author

gramian commented Feb 2, 2025

I did not check what "bucketName" holds. A readable name, ie property name or user-assigned bucket name would work.

@gramian
Copy link
Collaborator Author

gramian commented Feb 3, 2025

I should further clarify: if a type has many properties and some properties maybe even manually named buckets, then it is very tedious to test which property or bucket causes this warning. So naming the responsible property or bucket would be most helpful.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants