When the number of attempting to restart receiver greater than 0,spark do nothing in 'else'

[liuxianjiao]

When the number of attempting to restart receiver greater than 0,spark do nothing in 'else'.So I think we should log trace to let users know why.

[AmplabJenkins]

Can one of the admins verify this patch?

[srowen]

I don't think this is worthwhile.

[liuxianjiao]

@srowen
Thanks for your reply!We can let users know the theory of the restart of receiver by this PR.To be honest,the 'else' was redundancy,so I improve it by logtrace. To say the least,if this PR is not worthwhile and the 'else' do nothing,can we remove it.

[jerryshao]

I'm -1 on this PR. This PR actually fix nothing instead of adding one trace log, also usually user will not enable trace log, so this one line fix is not so helpful.

You can find many place in Spark where it has no "else" branch, are you going to fix them all?

[liuxianjiao]

@jerryshao so,what's the meaning of these empty 'else'?

[jerryshao]

NVM, I mean in the Spark code there're some intended empty "else" branch, are you going to add trace log to them all?