{"push_id":850089261,"size":1,"distinct_size":1,"ref":"refs/heads/jcr-change-initialize","head":"8d13fc3ca34d49899bfc63593b7d4fb96f62a99d","before":"b072d2279c1555864c914f49e2b02aef2e52dad4","commits":[{"sha":"8d13fc3ca34d49899bfc63593b7d4fb96f62a99d","author":{"email":"4ce87f5ca5596825d366b739cd5c4750d74e0217@gmail.com","name":"Collin Redding"},"message":"Update `DatWorkerPool` to be initialized with options hash\n\nThis updates `DatWorkerPool` to be initialized with an options\nhash. This is better than a list of ordered arguments because its\nmore flexibile and can expand in the future. This is also setup\nfor allowing a queue and worker class to be passed when its\ninitialized. For now, the do work proc is being passed, which is\nnot the best interface but this will be changed once a worker\nclass can be passed.\n\nThis also changes the worker pool to not take a debug flag but\ninstead take a logger. This provides more flexibility for logging\nand doesn't force only logging to stdout. The default is a null\nlogger which is a no-op (the methods do nothing) instead of using\n`/dev/null` (because this isn't compatible on all systems).\n\nThis also adds validation for the number of workers to ensure it\nis valid. This was mentioned as a nice cleanup when it took min and\nmax workers.\n\nFinally, this cleans up the dat worker pool unit tests to use the\nnew interface, adds comments for future changes related to passing\na queue and worker class and moves some of them to system tests.\nThis is part of updating the tests to more clearly divide unit and\nsystem and is also setup for passing a queue and worker class.","distinct":true,"url":"https://api.github.com/repos/redding/dat-worker-pool/commits/8d13fc3ca34d49899bfc63593b7d4fb96f62a99d"}]}