Print

Print


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