Skip to content

STORM-307: reset LocalState if files are corrupt#282

Merged
asfgit merged 2 commits intoapache:masterfrom
wurstmeister:STORM-307
Nov 4, 2014
Merged

STORM-307: reset LocalState if files are corrupt#282
asfgit merged 2 commits intoapache:masterfrom
wurstmeister:STORM-307

Conversation

@wurstmeister
Copy link
Contributor

  • allow supervisor to start even if LocalState files are empty

* allow supervisor to start if LocalState was corrupted

Change-Id: I3dbf957dd2e37c9c6be39dab29a9a71e99b7a2c3
Change-Id: Iffa4609ee9b8a0afbd437d2b2ee802cc83773a0a
@wurstmeister
Copy link
Contributor Author

Thanks @HeartSaVioR! should be fixed now.
I was wondering if we actually need this retry logic. i.e. if we can just reset the state whenever there is an IOException (rather than not starting up and users having to manually remove state files).

I am not sure why the retry logic was implemented this way, so I picked out the empty file case because that's been reported multiple times (incl the workaround of resetting the state).

@ptgoetz
Copy link
Member

ptgoetz commented Oct 24, 2014

+1

2 similar comments
@Parth-Brahmbhatt
Copy link
Contributor

+1

@clockfly
Copy link
Contributor

clockfly commented Nov 4, 2014

+1

@asfgit asfgit merged commit ccde233 into apache:master Nov 4, 2014
knusbaum pushed a commit to knusbaum/incubator-storm that referenced this pull request Feb 11, 2015
[BUG:7137863] [YSTORM-238] Disable UGI.loginUserFromSubject for hadoop AutoTGT login
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.

5 participants