diff options
author | Jorge Ruesga <jorge@ruesga.com> | 2017-05-20 07:58:37 +0100 |
---|---|---|
committer | Harry Youd <harry@harryyoud.co.uk> | 2017-05-20 07:59:22 +0100 |
commit | 5ce7272a0d3d46b484159416cca4ba9c39a15eeb (patch) | |
tree | 733b09f8e38ed4348c27a2a598fedde2b7cf62a3 | |
parent | 477f0bae68c2b34388030b5ba76e7241df9fa203 (diff) | |
download | contributors-cloud-generator-5ce7272a0d3d46b484159416cca4ba9c39a15eeb.tar.gz contributors-cloud-generator-5ce7272a0d3d46b484159416cca4ba9c39a15eeb.tar.bz2 contributors-cloud-generator-5ce7272a0d3d46b484159416cca4ba9c39a15eeb.zip |
Fix cause of Null Pointer
Commit 7d0f35240db7f7f391c791d125165e59fc5589ac fixed
the symptom but not the cause of the problem. Let's
catch null names before parsing them later
Change-Id: I65ebe341a734a56f7cbce3999b5db8f7e1176d8a
-rw-r--r-- | source/src/main/java/CloudGenerator.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/src/main/java/CloudGenerator.java b/source/src/main/java/CloudGenerator.java index 32bf7d9..2390533 100644 --- a/source/src/main/java/CloudGenerator.java +++ b/source/src/main/java/CloudGenerator.java @@ -315,7 +315,7 @@ public class CloudGenerator { private static void generateCloudDataFromGerrit(Connection conn) throws Exception { Statement st = conn.createStatement(); st.execute("insert into cloud_data (commits, id, name, username, filter) " + - "select count(*) commits, a.id, a.name, a.username, f.filter from ( " + + "select count(*) commits, a.id, ifnull(a.name, a.username), a.username, f.filter from ( " + "select (select e.id from emails e where e.email = c.author_email) id, changeId from gerrit_commits c " + "where c.subject <> 'Automatic translation import' " + ") c, accounts a, " + |