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

    https://github.com/apache/spark/pull/21180#discussion_r184870575
  
    --- Diff: python/pyspark/serializers.py ---
    @@ -523,7 +523,21 @@ def namedtuple(*args, **kwargs):
             for k, v in _old_namedtuple_kwdefaults.items():
                 kwargs[k] = kwargs.get(k, v)
             cls = _old_namedtuple(*args, **kwargs)
    -        return _hack_namedtuple(cls)
    +
    +        import sys
    +        f = sys._getframe(1)
    --- End diff --
    
    Yea but thing is, that the doc says this not guaranteed although most of 
Python implementations look having it - there's a risk here we should take (it 
could be broken in a specific implementation of Python although it sounds 
unlikely). Is there any other way to avoid this? 


---

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

Reply via email to