Commit graph

5199 commits

Author SHA1 Message Date
Yuce Tekol
1e30a9372d
Disabled unquoting for single quoted attribute values; updated test 2018-11-06 22:58:37 +03:00
Yuce Tekol
e9c1606a9d
Merge branch 'master' into allow-backslash-cr-in-pql-strings 2018-11-06 22:49:28 +03:00
Yuce Tekol
b158f1c378
unquote attribute values 2018-11-06 22:47:09 +03:00
alanbernstein
2e7d44ba29
Merge pull request #1722 from pilosa/minor-docs-updates
Minor docs updates
2018-11-06 11:43:29 -06:00
alanbernstein
7e6a2e41f6
Merge branch 'master' into minor-docs-updates 2018-11-06 11:30:13 -06:00
Yuce Tekol
56ee728724
Merge branch 'master' into allow-backslash-cr-in-pql-strings 2018-11-06 14:12:36 +03:00
Yuce Tekol
2a8f2c2026
minor grammar fix 2018-11-06 14:09:32 +03:00
Travis Turner
2ddde5fac6
Merge pull request #1719 from travisturner/import-shard-forward
forward imports to non-coordinator shards
2018-11-05 21:49:06 -06:00
Travis Turner
063caed8d1
forward imports to non-coordinator shards 2018-11-05 17:39:42 -06:00
Alan Bernstein
be5f8c923d Remove 'view' from nav 2018-11-05 12:34:08 -06:00
seebs
dde7551b88
Merge pull request #1712 from seebs/seebs/btree
Ensure btree comparison doesn't fail for smallish N

(Which is to say, differences between values exceeding the max value
of int.)
2018-11-05 11:53:41 -06:00
Alan Bernstein
497c541aff Remove outdated anchor link 2018-11-05 11:37:03 -06:00
Alan Bernstein
100c780704 Switch data model diagrams to png 2018-11-05 11:36:55 -06:00
Seebs
456c092930 enterprise/b: Ensure 64-bit range for btree keys.
The cmp() function used to compare btree keys was using the
trick of comparing unsigned values by coercing the result
of subtraction to a signed type. This is fine as long as
the range of differences never actually exceeds the limits
of the signed type. For instance, with int64, as long as
the magnitude of the difference is under 2^63 or so, it
works reasonably well.

Plain int, however, can be a 32-bit type, at which point
the magnitude of difference needed to break it is only
2^31 or so.

Subtraction and type conversion is enough cheaper than
branches that this is worth preserving, but it's worth
preserving by switching to an explicit int64 for the
operation and return type. This will not actually affect
performance except for people using the btree code on
32-bit machines, so it probably won't ever matter.
Which may also be true of the potential wrong answers,
but "might be slower" is a better risk than "might
crash".
2018-11-05 10:45:17 -06:00
seebs
3a2c2c227d
Merge pull request #1711 from seebs/seebs/method
Drop now-superfluous methodNotAllowedHandler
2018-11-05 09:38:05 -06:00
Matt Jaffee
6d821afed9
add note on skipped cluster test 2018-11-02 16:13:03 -05:00
Matt Jaffee
ec3982c0ff
pull out advertise URI changes 2018-11-02 16:10:39 -05:00
Matt Jaffee
02aee931a1
few small fixes 2018-11-02 16:03:50 -05:00
Matt Jaffee
d86e9ed3ea
add docker based cluster tests, remove proxy stuff, fix advertise 2018-11-02 14:41:42 -05:00
Yuce Tekol
3cb3a065dd
allow backlash, carriage return in pql strings 2018-11-01 21:56:34 +03:00
Seebs
ce815471fe Drop now-superfluous methodNotAllowedHandler
Long ago, the maintainers of gorilla/mux concluded that
it was a "wontfix" to return StatusMethodNotAllowed instead
of StatusNotFound for a method mismatch. Pilosa had a
workaround for this for the most common case (GET requests
to /index/{index}/query), and a TODO to address the other
cases.

While browsing the go-pilosa client code, I noticed that there
is a test for roaring import support which relies on getting
StatusMethodNotAllowed.  But how can this work, if gorilla/mux
doesn't do that?

Answer: They started doing it in mid-2017, apparently:
	https://github.com/gorilla/mux/issues/271

Dropping this code changes the exact text of the message
produced for that one case, but not the status code,
and makes the behavior less confusing.
2018-11-01 12:59:58 -05:00
Matthew Jaffee
5d99f70589
Merge pull request #1707 from dene14/docker-extra-tools
Add base system, curl and jq for debug and checks.
2018-10-31 09:34:40 -05:00
Denis Boulas
e80d5c5e56 Add base system, curl and jq for debug and checks. 2018-10-27 03:20:43 +03:00
Matt Jaffee
ec08930c30
Merge branch 'master' into cluster-tests 2018-10-26 16:14:00 -05:00
Matt Jaffee
62c1bfe90e
implement MustNewClusterWithProxy and drop/undrop for partitioning 2018-10-26 16:13:20 -05:00
Matt Jaffee
af7ece74fd
test drop and undrop in udproxy. 2018-10-26 09:17:26 -05:00
Matthew Jaffee
0fc08c696a
Merge pull request #1647 from jaffee/new-rows-iterate
copy non roaring-import code from Todds's row-iterate PR
2018-10-25 09:49:48 -05:00
Matt Jaffee
a0f4522b6a
fix up linter issues in new groupby/rows tests 2018-10-25 08:35:09 -05:00
Matt Jaffee
30590c83bd
Merge branch 'master' into new-rows-iterate 2018-10-25 08:14:58 -05:00
Matt Jaffee
21c35e6861
wrap errors, fix comment, add test 2018-10-24 17:18:39 -05:00
Matt Jaffee
671420f31d
add custom json marshal for FieldRow 2018-10-24 17:11:45 -05:00
Matt Jaffee
28591b4a91
WIP: figure out why config.Gossip.Port is 0 after cluster start 2018-10-24 16:56:32 -05:00
Matt Jaffee
f643487ce0
add initial UDP proxy code to support proxying/partitioning memberlist 2018-10-24 16:56:07 -05:00
Travis Turner
79bf01f79a
Merge pull request #1699 from travisturner/import-clear
add `clear` functional option for imports
2018-10-24 16:15:49 -05:00
Travis Turner
e69a8f3db4
handle clear flag on import roaring (http package) 2018-10-24 15:16:08 -05:00
Travis Turner
f02605a528
consolidate ImportOptions setup. use url.Values{}. fix comments. 2018-10-23 17:37:57 -05:00
Travis Turner
318e588b4d
document the --clear flag for imports 2018-10-23 17:37:57 -05:00
Travis Turner
f29b6e79ff
add import clear test coverage to http package 2018-10-23 17:37:57 -05:00
Travis Turner
5fb6ef224c
add clear support for ImportRoaring 2018-10-23 17:37:57 -05:00
Travis Turner
a7a15c64a2
support clear imports to int fields. fix bug in fragment.sum 2018-10-23 17:37:57 -05:00
Travis Turner
caf8e06712
add clear functional option for imports 2018-10-23 17:37:57 -05:00
Matthew Jaffee
dc3fbe4afa
Merge pull request #1700 from jaffee/import-docs
add more import docs
2018-10-23 14:32:52 -05:00
Matt Jaffee
0ab3ee7c73
Merge remote-tracking branch 'origin/master' into import-docs 2018-10-23 14:23:18 -05:00
Matt Jaffee
376c2d61ff
address CR feedback 2018-10-23 13:50:35 -05:00
Yuce Tekol
d127a83b1e
Merge pull request #1698 from yuce/1680-update-api-docs
Updated API docs
2018-10-23 21:46:17 +03:00
Matthew Jaffee
4eed160b2d
wip on adding import docs 2018-10-23 11:49:55 -05:00
Matt Jaffee
8db7a78a93
unpushed WIP on cluster-tests 2018-10-23 11:10:24 -05:00
Matthew Jaffee
ad26eeca1b
Merge pull request #5 from travisturner/new-rows-iterate-translated
translate column argument in Rows() query
2018-10-23 08:19:52 -05:00
Travis Turner
9fee746e76
translate column argument in Rows() query 2018-10-22 17:41:59 -05:00
Yuce Tekol
5060e6ee47
updated 2018-10-22 17:24:21 +03:00