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

table created without wordforms #1742

Closed
tomatolog opened this issue Jan 18, 2024 · 2 comments
Closed

table created without wordforms #1742

tomatolog opened this issue Jan 18, 2024 · 2 comments
Assignees
Labels

Comments

@tomatolog
Copy link
Contributor

the statements

drop table if exists index; create table index(f text) wordforms='C:/missed.txt';

finishes well without any warning or error but the resulting index got created without wordforms settings.

It should fails with the error that wordforms file missed.

@tomatolog tomatolog self-assigned this Jan 18, 2024
@sanikolaev sanikolaev added the bug label Jan 19, 2024
@tomatolog
Copy link
Contributor Author

that case is due to the wordforms = path/to/dict*.txt as the code that checks for files produces empty files list and that consider the valid case.
In the manual there is no examples of what consider wrong.

It could be better to provide the rules to such file expansion to make sure what form wordforms use - plain file or expansion of the file names.
And also make sure is empty list of files from the expansion is the valid case or not?

@tomatolog
Copy link
Contributor Author

since c9e61f4 if wordforms is the path to the file (without the wildcard file mask) and daemon can not open that file create table fails with the error message about the missed file

@sanikolaev sanikolaev added the rel::upcoming Upcoming release label Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants