Hi Jacek, changes are OK, one minor comment left on PR. Cheers, Andy From: Becla, Jacek Sent: Monday, December 07, 2015 8:27 PM To: Salnikov, Andrei A. <[log in to unmask]> Cc: qserv-l <[log in to unmask]> Subject: Re: [QSERV-L] DM-4348 merged Hi Andy Great work! I scanned your code and I have a few tiny tweaks, can you have a look at https://github.com/lsst/qserv/pull/164 ? Thanks Jacek On Dec 7, 2015, at 5:40 PM, Salnikov, Andrei A. <[log in to unmask]<mailto:[log in to unmask]>> wrote: Hi all, don't be surprised when you switch to qserv master and do not see qserv-czar service any more. Czar is not dead but it has been merged into mysql-proxy. Everything is supposed to function as before, except that for running integration tests you also need the tip of the master for qserv_testdata repo. Let me know if you see any interesting issues with new code. Cheers, Andy ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the QSERV-L list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1 ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the QSERV-L list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1