[Issue 496] Support for correct reconnections limit in consumer

[omnilight]

Fixes #496

Motivation

Ability to set maximum number of reconnections to broker for consumer without consumer been stuck

Modifications

• Added new method Closed() <-chan struct to consumer interface to be able to control if consumer is closed when using its Chan() method
• For partitioned consumer added call of the parent's Close() method in case when number of reconnections is limited. This fixes problem when consumer stuck in disconnected state.

Verifying this change

This change added tests and can be verified as follows:

• Added test for consumer closing on reaching max reconnection count
• Added test for new Closed() method

Does this pull request potentially affect one of the following parts:

• Dependencies (does it add or upgrade a dependency): (no)
• The public API: (no)
• The schema: (no)
• The default values of configurations: (no)
• The wire protocol: (no)

Documentation

• Does this pull request introduce a new feature? (no)
• If yes, how is the feature documented? (not documented)

[wolfstudy]

@omnilight Thanks for work on this, when reconnectToBroker fails, we should close the parentConsumer object. This pr is for the consumer side. Will the producer have the same problem?

[omnilight]

Added fix to prevent possible panic when pulsar returns error in consumer initialization step

[omnilight]

@wolfstudy Hi, could you please review this PR?

[clawfinger]

Hello, do we have any ETA on this issue? Is it still planned for 0.8.0? Started using this lib recently and need this functionality badly

[rueian]

I also think this PR is important. I frequently find my consumer code stucked without knowing the underlying maximum number of reconnection had been reached.

[omnilight]

@wolfstudy Could you please release this fix within the 0.9.0 release?

[omnilight]

@wolfstudy Sorry for annoying you, but is it possible to include this PR into next release?

[omnilight]

@wolfstudy Hi! Any changes should be made to accept this pr?

[omnilight]

Hello!

Any chance this PR will be accepted?

@RobertIndie sorry for dm

[RobertIndie]

Hello!

Any chance this PR will be accepted?

Hi @omnilight This PR looks good to me. But could you help resolve the conflicts? After that, I will review it again. Thanks.

[omnilight]

But could you help resolve the conflicts? After that, I will review it again. Thanks.

Yep, will fix that soon

[omnilight]

@RobertIndie Hi! I've rebases on latest master, please take a look