Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9532 )
Change subject: Add missing brace to example JSON ...................................................................... Patch Set 1: (2 comments) Thanks Alex for the review. Please see PS2. http://gerrit.cloudera.org:8080/#/c/9532/1/bin/compare_branches.py File bin/compare_branches.py: http://gerrit.cloudera.org:8080/#/c/9532/1/bin/compare_branches.py@36 PS1, Line 36: { "hash": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", "comment": "...",} > }, Done http://gerrit.cloudera.org:8080/#/c/9532/1/bin/compare_branches.py@36 PS1, Line 36: { "hash": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", "comment": "...",} > isn't it "{," Done -- To view, visit http://gerrit.cloudera.org:8080/9532 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ieabf181a6e052790ebe7034a1e4dd1632644fd8e Gerrit-Change-Number: 9532 Gerrit-PatchSet: 1 Gerrit-Owner: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Comment-Date: Wed, 07 Mar 2018 20:00:15 +0000 Gerrit-HasComments: Yes