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

    https://github.com/apache/spark/pull/21383#discussion_r190605953
  
    --- Diff: python/pyspark/tests.py ---
    @@ -1246,6 +1277,25 @@ def test_pipe_unicode(self):
             result = rdd.pipe('cat').collect()
             self.assertEqual(data, result)
     
    +    def test_stopiteration_in_client_code(self):
    +
    +        def stopit(*x):
    +            raise StopIteration()
    +
    +        seq_rdd = self.sc.parallelize(range(10))
    +        keyed_rdd = self.sc.parallelize((x % 2, x) for x in range(10))
    +        exc = Py4JJavaError, RuntimeError
    --- End diff --
    
    Got it. Makes sense. Let's add a single comment while we are here if you 
don't mind. Seems few changes are needed anyway.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to