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

Panic on Windows: possible related to integer compression. #4020

Closed
MichelDiz opened this issue Sep 18, 2019 · 1 comment · Fixed by #4066
Closed

Panic on Windows: possible related to integer compression. #4020

MichelDiz opened this issue Sep 18, 2019 · 1 comment · Fixed by #4066
Assignees
Labels
area/compression Related to compression of values, keys, and messages. kind/bug Something is broken. platform/windows Issues specific to Windows priority/P1 Serious issue that requires eventual attention (can wait a bit) status/accepted We accept to investigate/work on it.

Comments

@MichelDiz
Copy link
Contributor

What version of Dgraph are you using?

Dgraph version   : v1.1.0-45-g53916e18b
Dgraph SHA-256   : 4f9fcdd1819990de98f5a1f75569fd63163b67a8264e442b9cca97967df8f215
Commit SHA-1     : 53916e18b
Commit timestamp : 2019-09-17 19:30:15 +0530
Branch           : master
Go version       : go1.13

Have you tried reproducing the issue with the latest release?

No, cuz it's a PR testing.

What is the hardware spec (RAM, OS)?

Windows 10 pro

I can't reproduce it in Darwin. However, it seems to hurt the live loader performance.

Steps to reproduce the issue (command/config used to run Dgraph).

1 - Run zero
2 - Bulk load 21 mi RDF
2.1 - Start a single cluster/node (One Alpha, One zero).
3 - Live Load 21 mi RDF again (just to push it) with Live Loader's default settings.
4 - Repeat again the live load with the same setting until Alpha crash and live load closes the connection and throws an error.

If you decrease -c to 1

-c, --conc int    Number of concurrent requests to make to Dgraph (default 10)

This panic doesn't happen (or would take longer to happen)

Expected behaviour and actual result.

v1.1.0-45-g53916e18bLogs.txt

@MichelDiz MichelDiz added the investigate Requires further investigation label Sep 18, 2019
@MichelDiz MichelDiz changed the title Panic possible related to integer compression. Panic: possible related to integer compression. Sep 18, 2019
@mangalaman93 mangalaman93 changed the title Panic: possible related to integer compression. Panic on Windows: possible related to integer compression. Sep 18, 2019
@mangalaman93 mangalaman93 added area/compression Related to compression of values, keys, and messages. kind/bug Something is broken. platform/windows Issues specific to Windows status/accepted We accept to investigate/work on it. priority/P1 Serious issue that requires eventual attention (can wait a bit) and removed investigate Requires further investigation labels Sep 18, 2019
@mangalaman93
Copy link
Contributor

May be related to golang/go#23279

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/compression Related to compression of values, keys, and messages. kind/bug Something is broken. platform/windows Issues specific to Windows priority/P1 Serious issue that requires eventual attention (can wait a bit) status/accepted We accept to investigate/work on it.
Development

Successfully merging a pull request may close this issue.

3 participants