[KYLIN-2875] Cube e-mail notification Validation

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: v2.3.0
• Component/s: REST Service, Web
• Assignee: Julian Pan
• Reporter: Julian Pan
• Created: 18/Sep/17 10:34
• Updated: 02/Apr/18 13:42
• Resolved: 08/Dec/17 13:18

Description

Validation e-mail notification list avoid EmailException:

2017-03-28 20:01:56,453 INFO [pool-10-thread-8] execution.AbstractExecutable:250 : notify list:[[e-mail];[e-mail];, [e-mail]]
2017-03-28 20:01:56,453 ERROR [pool-10-thread-8] util.MailService:112 : Illegal semicolon, not in group
org.apache.commons.mail.EmailException: Illegal semicolon, not in group
at org.apache.commons.mail.Email.createInternetAddress(Email.java:541)
at org.apache.commons.mail.Email.addTo(Email.java:653)
at org.apache.commons.mail.Email.addTo(Email.java:636)
at org.apache.commons.mail.Email.addTo(Email.java:618)
at org.apache.kylin.common.util.MailService.sendMail(MailService.java:97)
at org.apache.kylin.common.util.MailService.sendMail(MailService.java:70)
at org.apache.kylin.job.execution.AbstractExecutable.notifyUserStatusChange(AbstractExecutable.java:251)
at org.apache.kylin.job.execution.DefaultChainedExecutable.onStatusChange(DefaultChainedExecutable.java:105)
at org.apache.kylin.engine.mr.CubingJob.onStatusChange(CubingJob.java:296)
at org.apache.kylin.job.execution.DefaultChainedExecutable.onExecuteFinished(DefaultChainedExecutable.java:89)
at org.apache.kylin.engine.mr.CubingJob.onExecuteFinished(CubingJob.java:292)
at org.apache.kylin.job.execution.AbstractExecutable.execute(AbstractExecutable.java:135)
at org.apache.kylin.job.impl.threadpool.DefaultScheduler$JobRunner.run(DefaultScheduler.java:133)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
Caused by: javax.mail.internet.AddressException: Illegal semicolon, not in group in string ``[e-mail];[e-mail];[e-mail];[e-mail];'' at position 16
at javax.mail.internet.InternetAddress.parse(InternetAddress.java:780)
at javax.mail.internet.InternetAddress.parse(InternetAddress.java:555)
at javax.mail.internet.InternetAddress.<init>(InternetAddress.java:91)
at org.apache.commons.mail.Email.createInternetAddress(Email.java:514)
... 15 more