Content-Disposition: inline
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 762

Hello Naiming:

First, I apologize for the delay in getting our comments back to you regarding the patch. In order to preserve formatting I have attached our
review as a text file.

In general, the patch mechanics are fine, but we would rather see the 
the relay port specification done per interface rather than as global parameter.  Details on this are included in the attachment.

To set expectations, these changes will not be included in our upcoming maintenance release, 4.3.6, due out in May '17. Rather they would go into our feature release, 4.4.0, which is due out later in the year, date is TBD.

Thank you for your efforts thus far.  If you have any comments or questions please let us know.

Sincerely,

Thomas Markwalder
ISC Software Engineering