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

单条插入任务超时时仍然有可能导致统计出错 #11

Closed
Linary opened this issue Oct 17, 2018 · 1 comment
Labels
bug

Comments

@Linary
Copy link
Contributor

@Linary Linary commented Oct 17, 2018

  1. TaskManager里面对singleTasks超时的情况没有做处理,这样会导致主线程遗漏一部分顶点插入量,然后如果这些singleTasks在边插入的过程中完成了,那会被统计到边的结果里面。

    由于超时可能导致有部分记录没有插入,而且用户也不知道,故取消超时时间,将插入机制修改为必须等待插入完成,这样也能避免统计出错。

  2. TaskManagershutdown部分忽略了singleExecutor

Linary added a commit that referenced this issue Oct 23, 2018
1. When a single insert task times out, it may still cause statistical errors.
2. TaskManager.shutdown() ignored singleExecutor

Fix #11

Change-Id: I9a468241f369f4c7e73a95ba9a5772dac90e0ce1
Linary added a commit that referenced this issue Oct 26, 2018
1. When a single insert task times out, it may still cause statistical errors.
2. TaskManager.shutdown() ignored singleExecutor

Fix #11

Change-Id: I9a468241f369f4c7e73a95ba9a5772dac90e0ce1
Linary added a commit that referenced this issue Oct 30, 2018
1. When a single insert task times out, it may still cause statistical errors.
2. TaskManager.shutdown() ignored singleExecutor

Fix #11

Change-Id: I9a468241f369f4c7e73a95ba9a5772dac90e0ce1
@Linary Linary added the bug label Nov 16, 2018
Linary added a commit that referenced this issue Nov 23, 2018
1. When a single insert task times out, it may still cause statistical errors.
2. TaskManager.shutdown() ignored singleExecutor

Fix #11

Change-Id: I9a468241f369f4c7e73a95ba9a5772dac90e0ce1
Linary added a commit that referenced this issue Nov 23, 2018
1. When a single insert task times out, it may still cause statistical errors.
2. TaskManager.shutdown() ignored singleExecutor

Fix #11

Change-Id: I9a468241f369f4c7e73a95ba9a5772dac90e0ce1
@javeme javeme closed this in #12 Nov 23, 2018
javeme added a commit that referenced this issue Nov 23, 2018
1. When a single insert task times out, it may still cause statistical errors.
2. TaskManager.shutdown() ignored singleExecutor

Fix #11

Change-Id: I9a468241f369f4c7e73a95ba9a5772dac90e0ce1
@Linary

This comment has been minimized.

Copy link
Contributor Author

@Linary Linary commented Nov 23, 2018

另一个会导致统计出错的问题:#3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.