Daniel >> consider renaming modules that deal with a query: >> add "q" in front, e.g., qcontrol (and qmerger?) > > Which others did you have in mind? just the two I mentioned, qcontrol, qmerger >> - thread.cc - pls add description to the header > What you really want is the mess of thread.cc and dispatcher.cc to be > re-thought and re-designed. Please capture in jira > No, testCppParser does not test the parser. Sadly, it is misnamed, much > like "the new parser" meant much more than just parsing. It tests the > whole query parsing, analysis, and rewriting framework. Do you have a > better name? It's an integration test of a few modules. Please capture in jira Thanks Jacek ######################################################################## 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