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

Доработки #16

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Доработки #16

wants to merge 7 commits into from

Conversation

dmpas
Copy link
Contributor

@dmpas dmpas commented Apr 25, 2017

Некоторые доработки из того, с чем возникали сложности в использовании.

The parser failed when meet comma, bracket or quote in comment.

Парсер сбивался, если в комментарии события были символы запятой,
фигурной скобки или кавычек.
When loading reference file in case of file format the data was cleared
but not filled if the reference file was not changed.

В случае работы с файловым журналом справочники очищались, но не
перезаполнялись.
In case of file log and set date filter The Service analyze date in a
filename and skips files with definitely unneeded data.

В случае отбора по дате и журнала в файловом формате программа
анализирует имя файла и отсекает те файлы, которые заведомо не содержат
данных, подходящих под отбор по дате.
When reading breaks in the middle of event, it will fail next iteration.
Fix is to allocate on a line that seems to be a start of event.
@dmpas
Copy link
Contributor Author

dmpas commented Jan 20, 2018

/cc @alekseybochkov

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 this pull request may close these issues.

1 participant