Hey Chamila, That's everything, great job getting this rolling!
-Randy On Wed, May 28, 2014 at 2:31 AM, Chamila Wijayarathna < cdwijayarat...@gmail.com> wrote: > Hi Roger and Randy, > I have updated [1] with the progress I have currently made. So now we have > cross language test suite testing nodejs servers and clients with nodejs, > C++ and java following protocols, transports and sockets. > > nodejs_protocols="binary compact json" > nodejs_transports="buffered framed" > nodejs_sockets="ip ip-ssl" > > @Randy, are we missing any protocols, transports or sockets here or is > this cover everything we have by now? > > @Roger, Can consider nodejs tests as completed? If not what else I need to > do on that? If this is ok for now, shall we move on to JavaScript or > Python? What do you think more suitable from those two? > > 1. > > https://docs.google.com/spreadsheet/ccc?key=0AolXxLs9J7hydEFMVmRmNl9RY3MwVFNtbnJSM2ZOSUE&usp=drive_web#gid=0<https://docs.google.com/spreadsheet/ccc?key=0AolXxLs9J7hydEFMVmRmNl9RY3MwVFNtbnJSM2ZOSUE&usp=drive_web#gid=0> > > Thank You! > > > On Wed, May 28, 2014 at 10:09 AM, Randy Abernethy < > randy.aberne...@gmail.com> wrote: > >> Hey Chamila, >> >> Node.js does not yet support processor events so you can skip that >> switch. Also >> Node.js is event driven and has no notion of worker threads, so you can >> also skip >> the workers switch. >> >> Best, >> Randy >> >> >> >> On Tue, May 27, 2014 at 10:37 AM, Chamila Wijayarathna < >> cdwijayarat...@gmail.com> wrote: >> >>> Hi, >>> I added patch to jira which add >>> '--port arg (=9090) Port number to listen' >>> to nodejs server at lib/nodejs/test/server.js and >>> '--host arg (=localhost) Host to connect' >>> '--port arg (=9090) Port number to connect' >>> to nodejs client at lib/nodejs/test/client.js. >>> These has been mentioned in http://thrift.apache.org/test/ and was >>> missing from nodejs test server and client. >>> I also noticed that java server is missing '--processor-events' and -n [ >>> --workers ], >>> nodejs server is missing '--processor-events' and -n [ --workers ] >>> and nodejs client is missing -n [ --testloops ]. >>> I'll study about processor events and testloops and try to add them. >>> For thread-pool workers, even though we have argument 'server-type', >>> servers we have in lib and test only creates simple servers. Even we give >>> something else, it will still create a simple server. So reading 'thread >>> pool workers' attribute from inputs seems to make no sense. Also in cross >>> language test suite, we are only using simple servers. So what should I do >>> regarding that. >>> Thank You! >>> >>> >>> On Tue, May 27, 2014 at 1:38 AM, Chamila Wijayarathna < >>> cdwijayarat...@gmail.com> wrote: >>> >>>> Hi Roger, >>>> I added java-nodejs and nodejs-java tests and attached patch at >>>> https://issues.apache.org/jira/browse/THRIFT-847. Please check if it >>>> is okay. >>>> I came up with few issues and I created following Jiras for them. >>>> https://issues.apache.org/jira/browse/THRIFT-2548 >>>> https://issues.apache.org/jira/browse/THRIFT-2547 >>>> https://issues.apache.org/jira/browse/THRIFT-2546 >>>> Is there anything more to add for nodejs tests? >>>> If this is enough for now, what should be the next step? >>>> >>>> Thank You! >>>> >>>> >>>> >>>> >>>> On Mon, May 26, 2014 at 6:50 PM, Chamila Wijayarathna < >>>> cdwijayarat...@gmail.com> wrote: >>>> >>>>> Hi Roger, >>>>> In the commit you have add 'compact' protocol also. >>>>> But compact protocol gives errors when testing with cpp. >>>>> But it seems like its not a problem of test suite, so we can fix them >>>>> later as you mentioned. >>>>> I'll add nodejs-java tests and ssl tests and attach the patch. >>>>> Thank You! >>>>> >>>>> >>>>> On Mon, May 26, 2014 at 2:31 AM, Roger Meier >>>>> <ro...@bufferoverflow.ch>wrote: >>>>> >>>>>> Hi Chamila >>>>>> >>>>>> Quoting Chamila Wijayarathna <cdwijayarat...@gmail.com>: >>>>>> >>>>>> Hi Roger, >>>>>>> >>>>>>> Upto now I did following on nodejs tests we discussed earlier. >>>>>>> >>>>>>> - Ran nodejs-nodejs, nodejs-java, java-nodejs tests manually >>>>>>> - Ran nodejs-cpp, cpp-nodejs, nodejs-nodejs tests manually with >>>>>>> compact >>>>>>> >>>>>>> protocol >>>>>>> >>>>>>> I came up with following observations. >>>>>>> >>>>>>> - We can add nodejs-nodejs, nodejs-java, java-nodejs tests to >>>>>>> test.sh in >>>>>>> >>>>>>> the same way we have implemented nodejs-cpp, cpp-nodejs tests. I >>>>>>> only >>>>>>> checked them manually, I couldn't ran them through test.sh since >>>>>>> I couldn't >>>>>>> figure out how to make 'node' command work on test.sh. Also >>>>>>> nodejs client >>>>>>> exists after 3 seconds, so I couldn't check if all tests give >>>>>>> success or >>>>>>> failure, is there any way to extend this limit? Node server seems >>>>>>> to fail >>>>>>> with multiException[1] tests in existing cpp tests, so I expect >>>>>>> same output >>>>>>> with other tests which use nodejs server. But I couldn't check it >>>>>>> since I'm >>>>>>> having my node client exists after 3 seconds. >>>>>>> >>>>>> >>>>>> yes, there seems to be an issue with multiException. >>>>>> >>>>>> I've also seen that we still miss nodejs-nodejs tests, I've added >>>>>> them. >>>>>> https://git-wip-us.apache.org/repos/asf?p=thrift.git;a=commitdiff;h= >>>>>> d9b331d2662ff8bbd14f04800ce5de56b21ba01a >>>>>> >>>>>> - Compact protocol seems to work ok with nodejs-nodejs, but when >>>>>>> cpp >>>>>>> >>>>>>> involves it gives some errors. So we need to look at what are the >>>>>>> issues >>>>>>> there. >>>>>>> >>>>>>> I would be very grateful if you can provide any help with problems I >>>>>>> have >>>>>>> mentioned above. >>>>>>> Should I create patch with this progress or shall we create patch >>>>>>> after we >>>>>>> solve these problems? >>>>>>> >>>>>> yes, Please provide patches also for combinations that are broken. >>>>>> We need full coverage of combinations and can fix the bugs in a >>>>>> second step. >>>>>> >>>>>> >>>>>> I also tried to run tests with 'ssl', since those options are >>>>>>> available in >>>>>>> node client and server at lib/nodejs/test, but I came up with errors >>>>>>> there. >>>>>>> Do we have 'ssl' for nodejs? >>>>>>> >>>>>> yes, this was added earlier this year. >>>>>> >>>>>> >>>>>> I also looked at existing JavaScript test on cross language test >>>>>>> suite, but >>>>>>> it seems to be very different from others. Can you give me some >>>>>>> information >>>>>>> on how it works? >>>>>>> >>>>>> Javascript test uses the javaserver from lib/js/test/src and phantomjs >>>>>> it's another story. >>>>>> >>>>>> >>>>>> thanks >>>>>> -roger >>>>>> >>>>>> >>>>>> >>>>>>> 1. testMultiException(success, test 3) >>>>>>> >>>>>>> events.js:72 >>>>>>> throw er; // Unhandled 'error' event >>>>>>> ^ >>>>>>> Error: read ECONNRESET >>>>>>> at errnoException (net.js:901:11) >>>>>>> at TCP.onread (net.js:556:19) >>>>>>> >>>>>>> Thank You! >>>>>>> >>>>>>> >>>>>>> On Thu, May 22, 2014 at 12:40 AM, Roger Meier < >>>>>>> ro...@bufferoverflow.ch>wrote: >>>>>>> >>>>>>> Hi Chamila >>>>>>>> >>>>>>>> >>>>>>>> Quoting Chamila Wijayarathna <cdwijayarat...@gmail.com>: >>>>>>>> >>>>>>>> Hi Roger, >>>>>>>> >>>>>>>>> >>>>>>>>> I think now we have a good automated test reporting? Do I have do >>>>>>>>> to any >>>>>>>>> modification on this? >>>>>>>>> >>>>>>>>> I'm fine with this for the moment. >>>>>>>> Everybody is now able to browse the tested combinations and results. >>>>>>>> >>>>>>>> >>>>>>>> If not what should I do next? Do you have any suggestions on tasks >>>>>>>> to >>>>>>>> >>>>>>>>> implement next. >>>>>>>>> Since in cross language test suite, it only tests nodejs with cpp, >>>>>>>>> I think >>>>>>>>> nodejs-nodejs, nodejs-java and java-nodejs tests are suitable as >>>>>>>>> next >>>>>>>>> task. >>>>>>>>> >>>>>>>>> yes, this makes sense. Do as many combinations as possible and add >>>>>>>> language by language. >>>>>>>> >>>>>>>> please check also that TestClient and TestServers follow the >>>>>>>> parameter >>>>>>>> signatures >>>>>>>> and return value definitions: http://thrift.apache.org/test/ >>>>>>>> >>>>>>>> What do you think? >>>>>>>> >>>>>>>>> >>>>>>>>> yes proceed with this. >>>>>>>> >>>>>>>> -roger >>>>>>>> >>>>>>>> >>>>>>>> Thank You! >>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> On Fri, May 16, 2014 at 9:17 PM, Chamila Wijayarathna < >>>>>>>>> cdwijayarat...@gmail.com> wrote: >>>>>>>>> >>>>>>>>> Patch attached at https://issues.apache.org/ >>>>>>>>> jira/browse/THRIFT-2534. >>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> On Fri, May 16, 2014 at 9:07 PM, Chamila Wijayarathna < >>>>>>>>>> cdwijayarat...@gmail.com> wrote: >>>>>>>>>> >>>>>>>>>> Sorry, status.html is attached here. >>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> On Fri, May 16, 2014 at 9:06 PM, Chamila Wijayarathna < >>>>>>>>>>> cdwijayarat...@gmail.com> wrote: >>>>>>>>>>> >>>>>>>>>>> Hi Roger, >>>>>>>>>>> >>>>>>>>>>>> I generated status.html I have attached with this. Please let >>>>>>>>>>>> me know >>>>>>>>>>>> what else should be added here. >>>>>>>>>>>> Thank You! >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> On Fri, May 16, 2014 at 1:19 PM, Chamila Wijayarathna < >>>>>>>>>>>> cdwijayarat...@gmail.com> wrote: >>>>>>>>>>>> >>>>>>>>>>>> Created a sub task and added patch. >>>>>>>>>>>> >>>>>>>>>>>>> https://issues.apache.org/jira/browse/THRIFT-2534 >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> On Fri, May 16, 2014 at 1:16 PM, Roger Meier < >>>>>>>>>>>>> ro...@bufferoverflow.ch >>>>>>>>>>>>> >wrote: >>>>>>>>>>>>> >>>>>>>>>>>>> yes, just creat a sub Task for THRIFT-847 >>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> thanks >>>>>>>>>>>>>> roger >>>>>>>>>>>>>> >>>>>>>>>>>>>> Quoting Chamila Wijayarathna <cdwijayarat...@gmail.com>: >>>>>>>>>>>>>> >>>>>>>>>>>>>> Hi Roger, >>>>>>>>>>>>>> >>>>>>>>>>>>>> I'll start working on that. >>>>>>>>>>>>>>> Shall I create a Jira for this and add patch for current >>>>>>>>>>>>>>> stuff >>>>>>>>>>>>>>> there? >>>>>>>>>>>>>>> Than You. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> On Thu, May 15, 2014 at 3:52 AM, Roger Meier < >>>>>>>>>>>>>>> ro...@bufferoverflow.ch >>>>>>>>>>>>>>> >wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Hi Chamila >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> generating this as an additional result out of test.sh is >>>>>>>>>>>>>>>> good >>>>>>>>>>>>>>>> first >>>>>>>>>>>>>>>> step. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> attached is a template for the second step: >>>>>>>>>>>>>>>> git apply test_status.diff >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> it uses jquery datatables, a simple and useful table control >>>>>>>>>>>>>>>> we can add filter functions to the table, etc. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> you can generate status.html >>>>>>>>>>>>>>>> or generate status.json and load the table within >>>>>>>>>>>>>>>> status.html from >>>>>>>>>>>>>>>> status.json >>>>>>>>>>>>>>>> (e.g. use https://datatables.net/manual/data#Objects as >>>>>>>>>>>>>>>> initialization >>>>>>>>>>>>>>>> method) >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> thanks >>>>>>>>>>>>>>>> roger >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Quoting Chamila Wijayarathna <cdwijayarat...@gmail.com>: >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Hi Roger, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> As we discussed earlier I have changed test.sh to write >>>>>>>>>>>>>>>> results of >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> cross >>>>>>>>>>>>>>>>> language test run to a status.md file. I have attached a >>>>>>>>>>>>>>>>> sample >>>>>>>>>>>>>>>>> status.mdgenerated. >>>>>>>>>>>>>>>>> I observed that in commit "THRIFT-847 Test Framework >>>>>>>>>>>>>>>>> harmonization >>>>>>>>>>>>>>>>> across >>>>>>>>>>>>>>>>> all languages (commit 4edac7ff085e4bd28096fef3cf8234 >>>>>>>>>>>>>>>>> e4991544ff)" >>>>>>>>>>>>>>>>> you have >>>>>>>>>>>>>>>>> wrote results of failed test to an error.log file. Are we >>>>>>>>>>>>>>>>> planning >>>>>>>>>>>>>>>>> to keep >>>>>>>>>>>>>>>>> both status.md and error.log or only one from them? >>>>>>>>>>>>>>>>> What do you think about the format of status.md? I am only >>>>>>>>>>>>>>>>> writing >>>>>>>>>>>>>>>>> success/failure, no any description in failure. Is there >>>>>>>>>>>>>>>>> any thing >>>>>>>>>>>>>>>>> else >>>>>>>>>>>>>>>>> needed to be added or is this enough for now? >>>>>>>>>>>>>>>>> Thank You! >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>> *Chamila Dilshan Wijayarathna,* >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> SMIEEE, SMIESL, >>>>>>>>>>>>>>>>> Undergraduate, >>>>>>>>>>>>>>>>> Department of Computer Science and Engineering, >>>>>>>>>>>>>>>>> University of Moratuwa. >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>> *Chamila Dilshan Wijayarathna,* >>>>>>>>>>>>>>> SMIEEE, SMIESL, >>>>>>>>>>>>>>> Undergraduate, >>>>>>>>>>>>>>> Department of Computer Science and Engineering, >>>>>>>>>>>>>>> University of Moratuwa. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>> -- >>>>>>>>>>>>> *Chamila Dilshan Wijayarathna,* >>>>>>>>>>>>> >>>>>>>>>>>>> SMIEEE, SMIESL, >>>>>>>>>>>>> Undergraduate, >>>>>>>>>>>>> Department of Computer Science and Engineering, >>>>>>>>>>>>> University of Moratuwa. >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> -- >>>>>>>>>>>> *Chamila Dilshan Wijayarathna,* >>>>>>>>>>>> SMIEEE, SMIESL, >>>>>>>>>>>> Undergraduate, >>>>>>>>>>>> Department of Computer Science and Engineering, >>>>>>>>>>>> University of Moratuwa. >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> -- >>>>>>>>>>> *Chamila Dilshan Wijayarathna,* >>>>>>>>>>> SMIEEE, SMIESL, >>>>>>>>>>> Undergraduate, >>>>>>>>>>> Department of Computer Science and Engineering, >>>>>>>>>>> University of Moratuwa. >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> -- >>>>>>>>>> *Chamila Dilshan Wijayarathna,* >>>>>>>>>> SMIEEE, SMIESL, >>>>>>>>>> Undergraduate, >>>>>>>>>> Department of Computer Science and Engineering, >>>>>>>>>> University of Moratuwa. >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>> >>>>>>>>> -- >>>>>>>>> *Chamila Dilshan Wijayarathna,* >>>>>>>>> SMIEEE, SMIESL, >>>>>>>>> Undergraduate, >>>>>>>>> Department of Computer Science and Engineering, >>>>>>>>> University of Moratuwa. >>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> -- >>>>>>> *Chamila Dilshan Wijayarathna,* >>>>>>> SMIEEE, SMIESL, >>>>>>> Undergraduate, >>>>>>> Department of Computer Science and Engineering, >>>>>>> University of Moratuwa. >>>>>>> >>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> *Chamila Dilshan Wijayarathna,* >>>>> >>>>> SMIEEE, SMIESL, >>>>> Undergraduate, >>>>> Department of Computer Science and Engineering, >>>>> University of Moratuwa. >>>>> >>>> >>>> >>>> >>>> -- >>>> *Chamila Dilshan Wijayarathna,* >>>> SMIEEE, SMIESL, >>>> Undergraduate, >>>> Department of Computer Science and Engineering, >>>> University of Moratuwa. >>>> >>> >>> >>> >>> -- >>> *Chamila Dilshan Wijayarathna,* >>> SMIEEE, SMIESL, >>> Undergraduate, >>> Department of Computer Science and Engineering, >>> University of Moratuwa. >>> >> >> > > > -- > *Chamila Dilshan Wijayarathna,* > SMIEEE, SMIESL, > Undergraduate, > Department of Computer Science and Engineering, > University of Moratuwa. >