Commit graph

5199 commits

Author SHA1 Message Date
Cody Soyland
d344876587 Increase gometalinter deadline 2018-08-02 10:59:29 -05:00
Cody Soyland
f6855fac62 Remove unused newMockReadCloser 2018-08-02 10:59:16 -05:00
Cody Soyland
dd7b5f517b
Merge branch 'master' into 1513-metalinter-megacheck 2018-08-02 10:29:47 -05:00
Cody Soyland
1774efaf19
Merge pull request #1571 from codysoyland/skip-flawed-translator-test
Skip flawed translator test
2018-08-01 11:23:05 -05:00
Cody Soyland
495b4259fd Skip flawed translator test 2018-08-01 10:57:56 -05:00
Cody Soyland
edf6fa9c67
Merge pull request #1568 from codysoyland/translator-test-sleep
Add more time to sleep in translator tests due to CI failures
2018-07-31 21:05:49 -05:00
Cody Soyland
c44bb65163 Add more time to sleep in translator tests due to CI failures 2018-07-31 20:39:05 -05:00
Travis Turner
5802ba37b2
Merge pull request #1552 from travisturner/translate-cluster-fix
Translate cluster fix
2018-07-31 16:39:33 -05:00
Travis Turner
2dd655f0e9
move sleep to be after replica sync 2018-07-31 16:01:42 -05:00
Cody Soyland
68dc6489ce
Merge branch 'master' into translate-cluster-fix 2018-07-31 15:13:31 -05:00
Cody Soyland
672563891e
Merge pull request #1558 from codysoyland/go-master-error
Use string prefix instead of equality so json error message will pass on all Go versions
2018-07-31 15:13:15 -05:00
Cody Soyland
ba3bda6ac2 Use string prefix instead of equality so json error message will pass on all Go versions 2018-07-31 14:16:17 -05:00
Cody Soyland
39a82091f2 Add sleep in tests to ensure writes make it to translate store 2018-07-31 14:11:24 -05:00
Cody Soyland
be60a91a58 Clarify error string for cases when reading from non-primary translate store when given non-existent key 2018-07-31 12:19:09 -05:00
Cody Soyland
0816ea8ecb Add sleep to test to wait for key replication 2018-07-31 12:11:21 -05:00
Cody Soyland
e79cd4b199 Add JSON parsing to translator test to verify keys 2018-07-31 11:40:30 -05:00
Cody Soyland
2739ce436c Merge remote-tracking branch 'travisturner/translate-cluster-fix' into translate-cluster-fix 2018-07-31 08:36:31 -05:00
Travis Turner
ac8f8aa2e4
remove incorrect mock implementation from tranlate test 2018-07-30 11:45:34 -05:00
Travis Turner
3ffafaca4a
ignore remote translation 2018-07-30 11:45:34 -05:00
Cody Soyland
5410dcd6b1
Fix errant references of server.primaryTranslateStore to server.translateFile 2018-07-30 11:45:34 -05:00
Travis Turner
45d89d7136
Merge pull request #1555 from travisturner/setrowattrs-with-rowkeys
update parser to handle row keys on SetRowAttrs()
2018-07-27 11:44:54 -05:00
Travis Turner
13c201d768
remove errant spew.Dump() 2018-07-27 11:15:22 -05:00
Travis Turner
dfd529b2f5
update parser to handle row keys on SetRowAttrs() 2018-07-27 10:50:04 -05:00
Matthew Jaffee
ada9857f35
Merge pull request #1551 from jaffee/1550-topn-doc
fix topn spec and example in docs
2018-07-26 09:30:01 -05:00
Matt Jaffee
f90a6dff8b
fix topn spec and example in docs 2018-07-26 08:12:50 -05:00
Cody Soyland
ca800c683e Add test for cluster translator 2018-07-24 14:57:06 -05:00
Cody Soyland
a427836a9a Fix errant references of server.primaryTranslateStore to server.translateFile 2018-07-24 10:06:20 -05:00
Travis Turner
3b0eb3068f
Merge pull request #1547 from travisturner/index-options
fix places where empty IndexOptions were being used
2018-07-23 21:57:37 -05:00
Travis Turner
f9a265f94d
fix places where empty IndexOptions were being used 2018-07-23 17:23:54 -05:00
Cody Soyland
dc50204846 Fix linter issues: unconvert 2018-07-20 11:51:55 -05:00
Cody Soyland
990780bd93 Fix linter issues: staticcheck (covered by megacheck, along with gosimple and unused) 2018-07-20 11:19:24 -05:00
Cody Soyland
e9523063b5 Fix linter issues: unused 2018-07-20 10:33:29 -05:00
Cody Soyland
f01d850b17 Fix linter issues: gosimple 2018-07-20 09:06:43 -05:00
Cody Soyland
c7e29b48f9
Merge pull request #1540 from codysoyland/1521-metalinter-vet
Fix linter issues: vet
2018-07-19 16:01:10 -05:00
Cody Soyland
ba0384fb52 Merge branch 'master' into 1521-metalinter-vet 2018-07-19 15:41:28 -05:00
Cody Soyland
cf6921fce2
Merge pull request #1542 from codysoyland/index-options-json
Add IndexOptions to IndexInfo json response
2018-07-19 15:37:29 -05:00
Cody Soyland
83372c0509
Merge branch 'master' into index-options-json 2018-07-19 15:37:08 -05:00
Travis Turner
383b758c41
Merge pull request #1524 from travisturner/mutex-field-type
add mutex field type
2018-07-19 15:17:05 -05:00
Cody Soyland
86185a7530
Merge branch 'master' into index-options-json 2018-07-19 14:51:43 -05:00
Cody Soyland
23e2961d65 Add field labels to struct literals 2018-07-19 14:49:50 -05:00
Matthew Jaffee
17118f86c2
Merge branch 'master' into mutex-field-type 2018-07-19 14:48:36 -05:00
Matthew Jaffee
0e9f9d7c0b
Merge pull request #1541 from jaffee/translate-test-race
fix race cond in translate_test
2018-07-19 14:43:33 -05:00
Cody Soyland
29eba09e1e Add IndexOptions to IndexInfo json response 2018-07-19 14:11:14 -05:00
Travis Turner
f95d0f93fd
remove mapVector implementation of the vector interface 2018-07-19 14:02:58 -05:00
Travis Turner
a733fd7f67
fix spelling mistake 2018-07-19 14:01:30 -05:00
Travis Turner
4202e67a9d
add docs for the mutex field type 2018-07-19 14:01:30 -05:00
Travis Turner
06f5d04f1c
swap out mapVector for rowsVector in mutex fields 2018-07-19 14:01:30 -05:00
Travis Turner
a07ed360ab
add mutex field type 2018-07-19 14:01:29 -05:00
Matt Jaffee
68f341ea9f
fix race cond in translate_test
implements locked Reader method on test.TranslateFile. This race condition is
only present in tests, and is due to a replica referring directly to a primary
instead of through an http.TranslateStore as it would in production. This patch
requires the replica to obtain a lock while accessing the primary, and requires
test.Reopen to obtain a lock when swapping out the pilosa.TranslateStore.

I was able to reproduce the race by running: "go test
-run=TestTranslateFile_PrimaryTranslateStore -race -count=10", and could not
reproduce it after this patch.

Somewhat unrelated, I came across a "fatal error: fault" triggered by trying to
print an open pilosa.TranslateFile. This seems to be related to the mmapped
".data" field. I wrote the test to document the issue, but I don't think it's
easily fixable.
2018-07-19 12:06:31 -05:00
Cody Soyland
de99b5720f Fix linter issues: vet 2018-07-19 11:37:50 -05:00