Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/7796#discussion_r36284932
  
    --- Diff: 
external/kafka/src/test/scala/org/apache/spark/streaming/kafka/DirectKafkaStreamSuite.scala
 ---
    @@ -381,3 +447,20 @@ object DirectKafkaStreamSuite {
         }
       }
     }
    +
    +private[streaming] class ConstantEstimator(rates: Double*) extends 
RateEstimator {
    --- End diff --
    
    Aah, the kafka project may not have spark tests as a test dependency. Dont 
worry about it then. Though I suggest making the test more deterministic like I 
am doing in my PR. You could copy the modified ConstantEstimator that I am 
using and use that to make the tests more deterministic. I can take care of 
deduping it later.


---
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