Skip to content

Elasticsearch #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 293 commits into
base: master
Choose a base branch
from
Open

Elasticsearch #6

wants to merge 293 commits into from

Conversation

@simonmulser
Copy link
Collaborator

@simonmulser simonmulser commented Aug 30, 2014

base for discussion and code review. it is very hard to split up in multiple commits. should i make commits like 'elasticsearch test helper'?

@simonmulser simonmulser force-pushed the elasticsearch branch 2 times, most recently from 1f17c24 to d92503c Compare September 4, 2014 06:49
@simonmulser
check if deploy time is string or time
a8ec986
@simonmulser
add occurrences count to mapping
0de4d89
@simonmulser
improve orderd by occurrence count test
93e7fec
@simonmulser
remove inspect function for easier debugging
6c9a71a
@simonmulser
use keyword arguments for occurrence search
20c50fb
@simonmulser
rename occurred_at to last_occurred_at
4a6fa82
@simonmulser
reuse find method from occurrence model
e401fb0
@simonmulser
refactor delete by query func to use keyword arg
55e686f
@simonmulser
refactor count func to use keyword args
7df78d8
@simonmulser
refactor search in es client to use keyword args
7aba7ac
@simonmulser
rename function find to get
34ea7db
@simonmulser
remove all from find method
226f042
@simonmulser
useless assert at the beginning of the test
3f7c063
@simonmulser
rename test
309fd42
@simonmulser
rearange class
8ba9566
@simonmulser
add to find the size argument
faca96f
@simonmulser
rename position to from
483e5a8
@simonmulser
add argument check for variable from
f08348d
@simonmulser
function to find occurrences since a date
a50326e
@simonmulser
remove unnecessary assert
21ac7e6
@simonmulser
remove all from method name
632fc3a
@simonmulser
move defautl sorting into find method
d408709
@simonmulser
check if empty
98404da
@simonmulser
reintroduce inspect method on models
23884f2
@simonmulser
fix wrong name of keyword argument
858f4ff
@simonmulser
rearrange method
1d68494
@simonmulser
use the word filters instead terms for keyword
1129464
simonmulser and others added 30 commits September 25, 2014 14:28
@simonmulser
break long lines
ebf3504
@simonmulser
remove unneeded self
c88bcdc
@simonmulser
use underscore to split long number
8ac1835
@simonmulser
add since last deploy link
abc508d
@simonmulser
use longer var names to make things clearer
7e10183
@simonmulser
remove abstract model class
8fc351a
@simonmulser
add if to avoid error when cgi-data is null
5067f0b
@simonmulser
add comment to explain empty rescue block
48b68e9
@simonmulser
making exception occurred more readable
5020fd3
@simonmulser
rename endpoint to elasticsearch_host
bd67b71
@simonmulser
remove mongodb leftovers
23e3423
@simonmulser
move utils into lib folder
09361c0
@simonmulser
dont abbreviate aggregation anymore
b6d8fd8
@simonmulser
remove unused function
a22b0fa
@simonmulser
fix pagination error when filtering
c50350e
@simonmulser
rename variable to more sensful name
3c4cf54
@simonmulser
split terms apart from filters
22012a0
@simonmulser
update export types
b044725
@stengaard
new naming
384340c
@stengaard
add helper script
2e6d562
@stengaard
fix spelling
f81b193
@stengaard
remove occurrences_count field. Use ES to get the value instead (lead…
539c87d 
…s to data race).
@stengaard
sanitize for nil terms
74fa7ce
@theflow
upgrade elasticsearch-extensions
be44350
@theflow
fix occurrences count aggregation
2205caa
@theflow
recreate the index instead of using delete_by_query
5ad74be
@theflow
replace delete_by_query with scan and bulk
1164b43 
delete_by_query is gone on Elasticsearch 2
@theflow
move this to the right place
0639751
@theflow
don’t need this anymore
579c1d4
@theflow
upgrade elasticsearch gem
ed33681
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@simonmulser @theflow @stengaard