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

    https://github.com/apache/spark/pull/21702#discussion_r199723478
  
    --- Diff: dev/create-release/releaseutils.py ---
    @@ -49,13 +49,16 @@
         print("Install using 'sudo pip install unidecode'")
         sys.exit(-1)
     
    +if sys.version < '3':
    +    input = raw_input
    --- End diff --
    
    Two downsides to that approach:
    1. We stick with the legacy Python syntax which will unnecessarily 
complicate our lives (and our diffs) [in 18 months](http://pythonclock.org)
    2. This approach reduces the linting errors from 10 down to just 2.


---

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

Reply via email to