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

    https://github.com/apache/spark/pull/21383#discussion_r190598641
  
    --- 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 --
    
    Hm .. can we just check one of explicit exception if it's not hard? 
Py4JJavaError or RuntimeError somehow sounds a bit two arbitrary exceptions ...


---

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

Reply via email to