Github user tdas commented on the pull request:

    https://github.com/apache/spark/pull/7600#issuecomment-125367702
  
    Just so that the comment to a hidden thread is not lost, I am repeating it 
here. For the `rateController: Option[RateController]` I propose this. 
    1. Let there be a object method RateController.isBackpressureEnabled, which 
will return true if spark.streaming.backpressure.enabled. 
    2. Class ReceiverInputDStream will create a controller instance of 
ReceiverRateController if the above method returns true. You may or may not 
wrap that check in an object method, ReceiverRateController.create(): 
Option[RateController]
    3. DirectKafkaRateController will create a conroller instance of 
DirectKafkaRateController in the same way.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to