CommonGarden / Grow-IoT

Software packages for smart growing environments.
http://commongarden.org/
Other
25 stars 6 forks source link

Elasticsearch syncing #302

Closed JakeHartnell closed 7 years ago

JakeHartnell commented 7 years ago

Searching is going to be important! Also with elastic search comes optional support for things like Kibana and Grafana which would help resolve #211.

https://hharnisc.github.io/2015/06/29/enhancing-meteor-with-elasticsearch.html#1 https://github.com/toystars/elasticsearch-sync/

aruntk commented 7 years ago

@JakeHartnell A basic implementation as in the doc you shared.

https://github.com/CommonGarden/Grow-IoT/tree/elastic-search

screen shot 2016-12-21 at 8 45 12 pm screen shot 2016-12-21 at 9 00 43 pm
JakeHartnell commented 7 years ago

Elastic search syncing is working great, but I think we broke something in the process:

selection_028

Console:

TypeError: exists is not a function(…)
111768Uncaught TypeError: BSON is not a constructor(…)
Server @ modules.js?hash=dc8f4ac…:111768Server @ modules.js?hash=dc8f4ac…:103892(anonymous function) @ modules.js?hash=dc8f4ac…:103145translateOptions @ modules.js?hash=dc8f4ac…:103142createServer @ modules.js?hash=dc8f4ac…:103165connect @ modules.js?hash=dc8f4ac…:103292MongoClient.connect @ modules.js?hash=dc8f4ac…:103082(anonymous function) @ modules.js?hash=dc8f4ac…:89376run @ modules.js?hash=dc8f4ac…:187074(anonymous function) @ modules.js?hash=dc8f4ac…:187088Collection._getCollection @ modules.js?hash=dc8f4ac…:89557Collection.getIndexes @ modules.js?hash=dc8f4ac…:89766exports.index @ modules.js?hash=dc8f4ac…:89000Queue @ modules.js?hash=dc8f4ac…:88870Connection.queue @ modules.js?hash=dc8f4ac…:88711meteorInstall.imports.api.elastic.queue.js @ queue.js:5fileEvaluate @ modules-runtime.js?hash=637cb12…:191require @ modules-runtime.js?hash=637cb12…:116Mp.import @ modules.js?hash=dc8f4ac…:314meteorInstall.imports.api.elastic.observe.js @ observe.js:1fileEvaluate @ modules-runtime.js?hash=637cb12…:191require @ modules-runtime.js?hash=637cb12…:116Mp.import @ modules.js?hash=dc8f4ac…:314meteorInstall.imports.api.elastic.index.js @ index.js:1fileEvaluate @ modules-runtime.js?hash=637cb12…:191require @ modules-runtime.js?hash=637cb12…:116Mp.import @ modules.js?hash=dc8f4ac…:314meteorInstall.imports.api.index.js @ index.js:1fileEvaluate @ modules-runtime.js?hash=637cb12…:191require @ modules-runtime.js?hash=637cb12…:116Mp.import @ modules.js?hash=dc8f4ac…:314meteorInstall.client.main.js @ main.js:1fileEvaluate @ modules-runtime.js?hash=637cb12…:191require @ modules-runtime.js?hash=637cb12…:116(anonymous function) @ app.js?hash=ec5167b…:42526
aruntk commented 7 years ago

@JakeHartnell Everything seems to be fine in my system. How can I recreate the error?

JakeHartnell commented 7 years ago

Hmmm, sorry, forgot steps to reproduce.

  1. Create a new thing (click the + button).
  2. Click 'Create test thing' or connect the test device.
aruntk commented 7 years ago

@JakeHartnell unable to recreate the issue. Could you check again or maybe try resetting the db?

JakeHartnell commented 7 years ago

Resetting the db worked! Thanks!

aruntk commented 7 years ago

@JakeHartnell Great. :). Could you please come online on gitter?