Commit graph

1738 commits

Author SHA1 Message Date
Michael Baird
1d05707026 Rename datadog/statsd_test.go to statsd/statsd_test.go 2017-05-19 09:43:38 -05:00
Michael Baird
769173c642 Rename datadog/statsd.go to statsd/statsd.go 2017-05-19 09:42:15 -05:00
Michael Baird
bd3814ba0c Rename datadog_test.go to statsd_test.go 2017-05-19 09:33:36 -05:00
Michael Baird
14dd1b3f35 Rename datadog.go to statsd.go 2017-05-19 09:33:00 -05:00
Ben Johnson
1c09af132e
Check for duplicate attributes under read lock on insert.
Changes the behavior of `AttrStore.SetAttrs()` to first verify
that the attributes haven't changed using a read-only lock before
obtaining a write lock to update the attributes.

Since the write lock serializes access, the previous insert time
sufferred lock contention when inserting a lot of duplicate
attributes. With the `RWMutex`, reads can be done in parallel so
multiple requests don't block each other.

Below is a simple benchmark showing the performance at 1, 5, & 10
goroutines:

	BenchmarkAttrStore_Duplicate          100000      137120 ns/op      8428 B/op       76 allocs/op
	BenchmarkAttrStore_Duplicate-5        100000      417459 ns/op      8431 B/op       76 allocs/op
	BenchmarkAttrStore_Duplicate-10       100000      139466 ns/op      8433 B/op       76 allocs/op

	BenchmarkAttrStore_Duplicate        20000000         703 ns/op       368 B/op        5 allocs/op
	BenchmarkAttrStore_Duplicate-5      100000000        213 ns/op       368 B/op        5 allocs/op
	BenchmarkAttrStore_Duplicate-10     100000000        223 ns/op       368 B/op        5 allocs/op
2017-05-19 07:35:50 -06:00
Alan Bernstein
75c59387f6 Skeleton for new autocomplete functionality 2017-05-18 18:34:23 -05:00
Alan Bernstein
39e46e2c48 Factor out autocompleter 2017-05-18 18:30:25 -05:00
Michael Baird
1c5f374c1a lower the StatsD sampling rate 2017-05-18 16:40:18 -05:00
Michael Baird
c41f695e74 more metrics tests 2017-05-18 16:36:22 -05:00
Cody Soyland
c9d5852817 Decrease heading depths now that H3 tags are supported by website CSS 2017-05-18 15:47:27 -05:00
Cody Soyland
b102b8cbc8 Use "note" class instead of blockquotes. This is needed to support the blockquotes in the style guide for the website. 2017-05-18 14:45:36 -05:00
Matt Jaffee
fb4651cdb8 fix 3 separate bugs in bitmapCountRange
in order of the diff:

1. When the start and end of the range fall in the same word, special handling
is needed to "mask" off the beginning and end of the word simultaneously to
avoid counting bits at the beginning or end of the word that aren't in the
range.

2. `i++` is needed at the end of the first partial word to avoid counting this
word in the next block.

3. the shift amount for right shifts is 64 - (end % 64) rather than just end
% 64. If end is (e.g.) 68, then 68 - 64 is 4 and we are only interested in the
first 4 bits of the word, so we must right shift by 60 bits, not 4 bits.
2017-05-18 12:04:37 -05:00
Michael Baird
58bfe8be80 Merge pull request #9 from linhvo/425-nodeID-stats
add nodeID tag
2017-05-18 11:12:42 -05:00
Michael Baird
815c647bea missing "." in StatsD prefix 2017-05-18 11:06:14 -05:00
Linh Vo
4328cc88cc add nodeID tag 2017-05-17 16:23:34 -05:00
tgruben
4a14e38ebe Merge pull request #549 from tgruben/conditional-asm
asm build options for non amd64
2017-05-17 11:16:21 -05:00
Cody Soyland
cb2c47a727 Move docs into main repo 2017-05-17 08:11:38 -05:00
Alan Bernstein
db256c91cc Only autocomplete on single match 2017-05-16 22:54:17 -05:00
Alan Bernstein
64a12c939c Fix typo in error message 2017-05-16 17:56:16 -05:00
Michael Baird
19ff6a2bd5 make fix after merge conflict 2017-05-16 16:45:53 -05:00
Michael Baird
7579567f59 return early when metrics are disabled 2017-05-16 15:28:18 -05:00
Michael Baird
d4e9e5cdbc cache StatsD recalculate, invalidate, and threshold counts 2017-05-16 15:21:26 -05:00
Michael Baird
abc8a672ed add "Pilosa" statsD constant prefix to event names 2017-05-16 15:12:44 -05:00
Linh Vo
4dd8221cd6 Merge pull request #555 from linhvo/552-max-slice
552 max slice
2017-05-16 14:41:54 -05:00
Travis Turner
6fd630f2ae Merge pull request #554 from travisturner/handleGetSliceMax-proto-bug
fix bug in `handleGetSliceMax` that was causing protobuf request to r…
2017-05-16 14:08:25 -05:00
Michael Baird
143cc14b1a Merge branch 'master' into metrics 2017-05-16 14:03:16 -05:00
Michael Baird
60cd24b368 set maxSlice StatsD gauge on get max slice 2017-05-16 14:00:54 -05:00
Linh Vo
81621e07b2 add tests 2017-05-16 13:12:08 -05:00
Travis
e31beb809b
fix bug in handleGetSliceMax that was causing protobuf request to return json 2017-05-16 12:26:25 -05:00
Linh Vo
f392853134 update sliceMaxByIndex by view 2017-05-16 11:11:11 -05:00
Michael Baird
b2343c5ce3 Stats uses the same logger output as the cluster 2017-05-16 10:52:10 -05:00
Linh Vo
f3d0aec46d Merge pull request #548 from linhvo/413-export-view
#413 set standard view as default for export command
2017-05-16 10:48:09 -05:00
Michael Baird
4a051900c0 Add sampling rate to StatsD functions, and fix data dog logging 2017-05-16 10:35:49 -05:00
Linh Vo
5de9a7d0d0 refactor view validator 2017-05-15 14:56:47 -05:00
Yuce Tekol
745888f5a3 Merge pull request #550 from yuce/add-coveralls
added goveralls
2017-05-15 22:32:41 +03:00
Ben Johnson
928ee49187 Merge pull request #551 from benbjohnson/515-topn-inverse
Add inverse TopN() support.
2017-05-15 12:53:15 -06:00
Cody Soyland
ef311c8932 Merge pull request #540 from codysoyland/makefile-updates
Makefile updates
2017-05-15 14:37:48 -04:00
Linh Vo
384329bc91 fix view always return error 2017-05-15 13:33:44 -05:00
Linh Vo
b635bdf970 #413 add tests and remove redundant word 2017-05-15 12:54:49 -05:00
Ben Johnson
0b8d935f1c
Add inverse TopN() support.
The TopN() call now supports an `inverse` boolean argument to
specify if the call should operate on the standard view or the
inverse view.
2017-05-15 10:35:48 -06:00
Yuce Tekol
c8d03ff625
added goveralls 2017-05-15 19:35:04 +03:00
Todd Gruben
fbae0f4d15 asm build options for non amd64 2017-05-15 11:31:41 -05:00
Linh Vo
e20b9075e2 #413 set standard view as default for export command 2017-05-15 11:17:48 -05:00
alanbernstein
5d603f623b Merge pull request #537 from alanbernstein/webui-help
Add syntax hints tooltip to WebUI
2017-05-12 16:33:16 -05:00
Cody Soyland
a8a15a6f79 Merge pull request #546 from graphaelli/checksum-error-message
provide details on glide checksum mistmach
2017-05-12 17:26:41 -04:00
Gil Raphaelli
76354c69e9 provide details on glide checksum mistmach
before:

Step 6/13 : RUN wget ${GLIDE} -O /go/glide.tar.gz -q     && tar xf /go/glide.tar.gz     && mv /go/linux-amd64/glide /go/bin     && [ "$(sha256sum /go/bin/glide | cut -d' ' -f1)" = "$GLIDE_HASH" ]     && cd /go/src/github.com/pilosa/pilosa     && make vendor     && CGO_ENABLED=0 go install -a -ldflags "$ldflags" github.com/pilosa/pilosa/cmd/pilosa
 ---> Running in e2a6fcdf5930
The command '/bin/sh -c wget ${GLIDE} -O /go/glide.tar.gz -q     && tar xf /go/glide.tar.gz     && mv /go/linux-amd64/glide /go/bin     && [ "$(sha256sum /go/bin/glide | cut -d' ' -f1)" = "$GLIDE_HASH" ]     && cd /go/src/github.com/pilosa/pilosa     && make vendor     && CGO_ENABLED=0 go install -a -ldflags "$ldflags" github.com/pilosa/pilosa/cmd/pilosa' returned a non-zero code: 1
make: *** [docker] Error 1

after:

Step 6/13 : RUN wget ${GLIDE} -O /go/glide.tar.gz -q     && tar xf /go/glide.tar.gz     && mv /go/linux-amd64/glide /go/bin     && echo "${GLIDE_HASH}  /go/bin/glide" | shasum -a 256 -c -     && cd /go/src/github.com/pilosa/pilosa     && make vendor     && CGO_ENABLED=0 go install -a -ldflags "$ldflags" github.com/pilosa/pilosa/cmd/pilosa
 ---> Running in 5aaaab387641
shasum: WARNING: 1 computed checksum did NOT match
/go/bin/glide: FAILED
The command '/bin/sh -c wget ${GLIDE} -O /go/glide.tar.gz -q     && tar xf /go/glide.tar.gz     && mv /go/linux-amd64/glide /go/bin     && echo "${GLIDE_HASH}  /go/bin/glide" | shasum -a 256 -c -     && cd /go/src/github.com/pilosa/pilosa     && make vendor     && CGO_ENABLED=0 go install -a -ldflags "$ldflags" github.com/pilosa/pilosa/cmd/pilosa' returned a non-zero code: 1
make: *** [docker] Error 1
2017-05-12 16:48:47 -04:00
Cody Soyland
3a1bcc353e Add release target in Makefile 2017-05-10 14:04:09 -05:00
Cody Soyland
6c11746ac8 Fix vendor target as to not call "glide install" on every invocation 2017-05-10 13:59:34 -05:00
Cody Soyland
86c254bdbd Keep ldflags in one place 2017-05-10 13:57:41 -05:00
Alan Bernstein
3eff46b07a Use default row/column labels 2017-05-09 21:22:45 -05:00