This is an automated email from the ASF dual-hosted git repository.
alexott pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git
The following commit(s) were added to refs/heads/master by this push:
new 6c5d2d2 [hotfix][dev] Fix error arising from using filter in Python 3
6c5d2d2 is described below
commit 6c5d2d22700fbb4240f4befdc75297b317277f1b
Author: Alex Ott <alex...@apache.org>
AuthorDate: Mon Jul 20 17:28:28 2020 +0200
[hotfix][dev] Fix error arising from using filter in Python 3
This should fix following error in `dev/merge_zeppelin_pr.py`:
```
Traceback (most recent call last):
File "dev/merge_zeppelin_pr.py", line 355, in <module>
resolve_jira_issues(title, merged_refs, jira_comment)
File "dev/merge_zeppelin_pr.py", line 293, in resolve_jira_issues
resolve_jira_issue(merge_branches, comment, jira_id)
File "dev/merge_zeppelin_pr.py", line 257, in resolve_jira_issue
default_fix_versions = set(map(lambda x: fix_version_from_branch(x,
versions).name, merge_branches))
File "dev/merge_zeppelin_pr.py", line 257, in <lambda>
default_fix_versions = set(map(lambda x: fix_version_from_branch(x,
versions).name, merge_branches))
File "dev/merge_zeppelin_pr.py", line 218, in fix_version_from_branch
return versions[0]
TypeError: 'filter' object is not subscriptable
```
---
dev/merge_zeppelin_pr.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/dev/merge_zeppelin_pr.py b/dev/merge_zeppelin_pr.py
index 557fc25..23b5d19 100755
--- a/dev/merge_zeppelin_pr.py
+++ b/dev/merge_zeppelin_pr.py
@@ -250,9 +250,9 @@ def resolve_jira_issue(merge_branches, comment,
default_jira_id=""):
versions = asf_jira.project_versions("ZEPPELIN")
versions = sorted(versions, key=lambda x: x.name, reverse=True)
- versions = filter(lambda x: x.raw['released'] is False, versions)
+ versions = list(filter(lambda x: x.raw['released'] is False, versions))
# Consider only x.y.z versions
- versions = filter(lambda x: re.match('\d+\.\d+\.\d+', x.name), versions)
+ versions = list(filter(lambda x: re.match('\d+\.\d+\.\d+', x.name),
versions))
default_fix_versions = set(map(lambda x: fix_version_from_branch(x,
versions).name, merge_branches))
for v in default_fix_versions:
