search

cfpb / grasshopper

CFPB's streaming batch geocoder
Creative Commons Zero v1.0 Universal
37 stars 13 forks source link

Dockerizing test-harness #203

Closed hkeeler closed 8 years ago

hkeeler commented 8 years ago

This PR takes care of a smattering of issues discovered while Dockerizing test-harness. It includes:

  1. Adds Dockerfile for test-harness.
  2. Sets a main class for test-harness.
  3. Fixes test-harness config for Elasticsearch host and port.
  4. Fixes test-harness sample data file format.
  5. New and modified log statements used when debugging issues in test-harness.
  6. Refactoring client to use Typesafe Config's envvar handling.
awolfe76 commented 8 years ago

@hkeeler - just a couple questions to help me understand.

awolfe76 commented 8 years ago

Thanks @hkeeler! Merging.

hkeeler commented 8 years ago

Thanks @awolfe76!