mediachain / mediachain-indexer

search, dedupe, and media ingestion for mediachain
33 stars 14 forks source link

Update datastore configuration for rpc datastore service #16

Closed yusefnapora closed 8 years ago

yusefnapora commented 8 years ago

This updates the configuration settings for the datastore, since we removed the direct dynamo db access in https://github.com/mediachain/mediachain-client/pull/37

Removes the aws environment variables from mc_config, and replaces with MC_DATASTORE_HOST and MC_DATASTORE_PORT_INT, which are passed to mediachain.datastore.rpc.set_rpc_datastore_config

If MC_DATASTORE_HOST is not set, uses MC_TRANSACTOR_HOST

parkan commented 8 years ago

I would just take out the datastore host to not be confusing, since it'll always run on transactor facade, right?

vyzo commented 8 years ago

We are currently running them together, but this may or may not change in the future :) I think having the ability to specify a different one, but defaulting to the same host as the Transactor Rpc service is the most generic solution.

autoencoder commented 8 years ago

Ah, we were editing the same files :)