Skip to content
This repository has been archived by the owner on Aug 5, 2019. It is now read-only.

Delayed hangup at start #10

Open
jwflory opened this issue May 16, 2015 · 4 comments
Open

Delayed hangup at start #10

jwflory opened this issue May 16, 2015 · 4 comments

Comments

@jwflory
Copy link
Member

jwflory commented May 16, 2015

Whenever a CrystalSpace server first starts up, sometimes the CrystalSpace plugin will hang during startup, subsequently delaying the entire startup process. The cause is currently unknown; however, it seems to happen in-between the first and last output of the following snippet.

[18:54:21 INFO]: [CrystalSpace] Enabling CrystalSpace v3.0.0-ALPHA-5
[18:54:21 INFO]: [CrystalSpace] The plugin will now start sending usage stats. You can disable this plugins/PluginMetrics/config.yml!
[18:54:34 INFO]: UUID of player jflory7 is 73402589-7b19-4cc6-97c0-7599c6e9a6d5
[18:55:07 INFO]: UUID of player Tethtibis is 79571497-1da1-488e-b33a-b4315694dfc1
[18:56:20 INFO]: UUID of player Tethtibis is 79571497-1da1-488e-b33a-b4315694dfc1
[18:56:28 INFO]: [CrystalSpace] Enabled version 3.0.0-ALPHA-5.

The code will need to evaluated to determine the issue and then optimize the code. Presently, we are looking at an almost two-minute initialization, which should not be happening.

@iffa
Copy link

iffa commented May 17, 2015

        try {
            Metrics metrics = new Metrics();
            metrics.beginMeasuringPlugin(this);
        } catch (IOException e) {
            // Fail silently
            MessageHandler.debugPrint(Level.WARNING, "Failed to contact Metrics (usage stats)");
        }

Metrics issue? It should be async, but if the hangup is indeed random, perhaps it depends on what the Metrics class is up to.

@jwflory
Copy link
Member Author

jwflory commented May 21, 2015

@iffa
Perhaps, it's actually likely something has changed in the Metrics setup since it was originally implemented. I'll dig through it a bit today.

@jwflory jwflory added this to the Initial Release milestone May 21, 2015
@iffa
Copy link

iffa commented Jun 2, 2015

Possibly fixed with up-to-date Metrics class

(see 49ed7e9)

@jwflory
Copy link
Member Author

jwflory commented Oct 7, 2015

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants