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

    https://github.com/apache/spark/pull/20838#discussion_r204091905
  
    --- Diff: dev/create-release/releaseutils.py ---
    @@ -149,7 +152,11 @@ def get_commits(tag):
                 if not is_valid_author(author):
                     author = github_username
             # Guard against special characters
    -        author = unidecode.unidecode(unicode(author, "UTF-8")).strip()
    +        try:               # Python 2
    +            author = unicode(author, "UTF-8")
    +        except NameError:  # Python 3
    +            author = str(author)
    +        author = unidecode.unidecode(author).strip()
    --- End diff --
    
    @BryanCutler I think we do need to specify it because it won't be unicode 
type in Python 2, we get these from run_cmd which call's Popen communicate 
which returns a regular string.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to