pelias/api - api - CogtoSrc

Commit Graph

Author	SHA1	Message	Date
Julian Simioni	0e8f4253c0	Return multiple results in place when osm node and way share an id It turns out the _type parameter to the Elasticsearch [multiget](https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-multi-get.html) API does not allow an array of possible values. We were depending on its ability to search multiple types to allow searching for OSM nodes and ways. But, since this doesn't work we essentially have to do it ourselves. There is also the problem that OSM nodes and ways share an ID space. So a gid such as `osm:venue:5` could in theory correspond to 2 records. It seems like the only nice thing to do in that case is return both results. This PR "unrolls" such queries. For example, in the case of `osm:venue:5`, the sanitisers will return the following array of objects to be turned into multiget queries: ``` [{ id: 5, types: ["osmway", "osmnode"] }] ``` Before, this would turn into a multiget query with only one entry, like this: ``` { "docs": [ { "_index": "pelias", "_type": [ " osmnode", "osmway" ], "_id": 5 } ] } ``` now it would look like this: { "docs": [ { "_index": "pelias", "_type": "osmnode", "_id": 5 }, { "_index": "pelias", "_type": "osmnode", "_id": 5 } ] } TLDR you might get back more records from /place than the number of ids you specified, but at least you will definitely get back what you are looking for.	9 years ago
Stephen Hess	cb7d22df91	switched ES lookup field from "type" -> "_type", fixes issue with /place lookup on Eiffel Tower gid	9 years ago
Julian Simioni	2505e92a62	Expect an array of types from _ids sanitiser This doesn't have any effect by itself but allows for the 3-part gid sanitiser to possibly return multiple types (i.e. in the case of osm:venue:1000)	9 years ago
Julian Simioni	aabca1569c	Fix old reference to /doc endpoint	9 years ago
Diana Shkolnikov	2b22d611bd	Fix lost confidence score	9 years ago
Peter Johnson	348d2e70e2	more conformance	9 years ago
Diana Shkolnikov	2eccfc5c9a	Fix unit test	9 years ago
Diana Shkolnikov	f7044d2435	Add content for `attribution` static endpoint Fixes #212	9 years ago
Diana Shkolnikov	db0e063d8c	Refactor controllers to be simple midddleware Refactor search and doc controllers to allow for post-processing middleware to handle geojson and response sending. Allows for a much more flexible routing scheme.	9 years ago
Julian Simioni	167547a6c1	Rename doc endpoint to place	9 years ago
Julian Simioni	2fce64f75a	Move files	9 years ago
Julian Simioni	4e74e9b681	Whitespace	9 years ago
Harish Krishna	e5f783567f	remove all things that involved suggester including query mixer keeping the suggest sanitizer for suggest/nearby wherein lat/lon are required	10 years ago
Harish Krishna	b9c4c673f5	Revert "removing searchType overhead" This reverts commit `ee4aa1b97a`.	10 years ago
Harish Krishna	ee4aa1b97a	removing searchType overhead	10 years ago
Diana Shkolnikov	99ee2d95a3	Change to readFileSync and update test to use proxyquire	10 years ago
Diana Shkolnikov	4f1b75ce94	Fix test name	10 years ago
Diana Shkolnikov	1d45b20268	Load markdown file into memory and check if client requested html	10 years ago
Harish Krishna	a91cb23ea4	adding a new file DOCS.md (same as https://github.com/pelias/api/wiki/API-Endpoints ), modifying controller/index.js to return text/html instead of application/json by default. Using markdown module to convert MD to HTML. Fixing tests (removed test/jsonp.coffee)	10 years ago
Harish Krishna	0e9af3b9b8	adding controller tests to test details=true test case	10 years ago
Harish Krishna	c99c5f712a	adding a parameter called details (defaults to false for all endpoint except reverse) + fixing tests	10 years ago
Peter Johnson	ff757decb4	deprecate 'type' property, see PR notes	10 years ago
Harish Krishna	ff9e9973fc	switching searchType from the default 'query_then_fetch' to 'dfs_query_then_fetch' because we have multiple shards in prod http://www.elasticsearch.org/blog/understanding-query-then-fetch-vs-dfs-query-then-fetch/	10 years ago
Harish Krishna	bcf823c519	cleanup + tests	10 years ago
Harish Krishna	fc7f6cb768	removing suggest_nearby, replacing suggest with suggest_multiple, commenting out a couple of tests for right now.	10 years ago
Harish Krishna	69a13e9105	fixing all the js lint related issues (lintified)	10 years ago
Harish Krishna	407c32e782	suggest_near_me to /suggest/nearby and suggest_near_me.js to suggest_nearby.js	10 years ago
Harish Krishna	886aa498f5	bringing back vanilla suggest (suggestions based on lat/lon/zoom only) - calling it suggest_near_me and adding a new endpoint (plus tests)	10 years ago
Harish Krishna	a53a6fd6d8	no query needed	10 years ago
Harish Krishna	be39fb5a8f	adding tests for the doc controller	10 years ago
Harish Krishna	f1d13c1337	consolidating suggest_poi and suggest_admin into one suggest endpoint. fixing tests (more tests to come)	10 years ago
Peter Johnson	6efa0aa510	expose id/type/layer	10 years ago
Peter Johnson	27e2c4e5b8	migrate outputGenerator from pelias/suggester-pipeline	10 years ago
Peter Johnson	6ddbe86ac5	perform mget during suggest. requires suggester output to be a gid	10 years ago
Harish Krishna	81a62482e7	query/suggest does not exist anymore. temp fix - using suggest_poi. changing controller/suggest to controller/suggest_poi as well	10 years ago
Peter Johnson	17134105d5	geojsonify /search and /reverse FTW	10 years ago
Peter Johnson	4b2ac09c7c	removed debug statement	10 years ago
Peter Johnson	613e7233af	flip lat/lon	10 years ago
Peter Johnson	42ac5f05eb	geojsonify suggest API	10 years ago
Peter Johnson	18f4f6b256	refactor/ add more tests	10 years ago
Peter Johnson	ee984c65b6	prettier reporting	10 years ago
Peter Johnson	3857109793	add unit test suite	10 years ago

45 Commits (coverage-test)