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

Realtime node cannot start back up if there is a "merged" directory on disk #123

Closed
cheddar opened this issue Apr 17, 2013 · 3 comments
Closed
Labels

Comments

@cheddar
Copy link
Contributor

cheddar commented Apr 17, 2013

The realtime nodes periodically merge data and store it in a local directory for pushing up to deep storage. When these directories are left lying around, the screw up the start up logic with exceptions like:

[2013-04-17 15:58:43,178] ERROR RuntimeException aborted realtime processing[rb_flow]: {class=com.metamx.druid.realtime.RealtimeManager, exceptionType=class java.lang.NumberFormatException, exceptionMessage=For input string: "merged"} (co
java.lang.NumberFormatException: For input string: "merged"
at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
at java.lang.Integer.parseInt(Integer.java:492)
at java.lang.Integer.parseInt(Integer.java:527)
at com.metamx.druid.realtime.plumber.RealtimePlumberSchool$1.bootstrapSinksFromDisk(RealtimePlumberSchool.java:377)
at com.metamx.druid.realtime.plumber.RealtimePlumberSchool$1.startJob(RealtimePlumberSchool.java:188)
at com.metamx.druid.realtime.RealtimeManager$FireChief.run(RealtimeManager.java:159)

We need to fix this in two ways:

  1. Make the restart logic resilient to this stuff
  2. Store the merged directories somewhere else so that they don't get in the way in the first place
@sixtus
Copy link
Contributor

sixtus commented Jun 7, 2013

Not involved, just trying to keep this up to date: There is a pull request now: https://github.com/metamx/druid/pull/158

@quode
Copy link
Contributor

quode commented Jun 7, 2013

Thanks sixtus. I forgot to link pull request #158 to this bug.

@cheddar
Copy link
Contributor Author

cheddar commented Jun 14, 2013

Merged. Thanks!

@cheddar cheddar closed this as completed Jun 14, 2013
b-slim pushed a commit to b-slim/druid that referenced this issue Jul 30, 2015
jihoonson pushed a commit to jihoonson/druid that referenced this issue Feb 3, 2021
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

3 participants