----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38448/#review99390 -----------------------------------------------------------
docs/src/site/twiki/FalconEmailNotification.twiki (line 3) <https://reviews.apache.org/r/38448/#comment156272> Recommend text change: Change "sending email based notification from Falcon when scheduled feed/process instance completed" To: "sending email notifications when scheduled feed/process instances complete." docs/src/site/twiki/FalconEmailNotification.twiki (line 8) <https://reviews.apache.org/r/38448/#comment156273> Will it confuse users that the "to" part looks like a literal e-mail address? Will people think that's the value they should use, rather than the intended recipient address? docs/src/site/twiki/FalconEmailNotification.twiki (line 14) <https://reviews.apache.org/r/38448/#comment156271> Recommend a text change: "specifies the address to send notifications to; multiple recipients may be provided as a comma-separated list" - Scott Preece On Sept. 17, 2015, 7:07 a.m., Peeyush Bishnoi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38448/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2015, 7:07 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1426 > https://issues.apache.org/jira/browse/FALCON-1426 > > > Repository: falcon-git > > > Description > ------- > > FALCON-1426: Documentation for Falcon Email notification > > > Diffs > ----- > > docs/src/site/twiki/EntitySpecification.twiki bf45598 > docs/src/site/twiki/FalconDocumentation.twiki 68ed2e2 > docs/src/site/twiki/FalconEmailNotification.twiki PRE-CREATION > > Diff: https://reviews.apache.org/r/38448/diff/ > > > Testing > ------- > > Yes. > > > Thanks, > > Peeyush Bishnoi > >
