aboutsummaryrefslogtreecommitdiffstats
path: root/rust
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' into cockroachx-attic-cockroachcockroachBryan Newbold2018-09-1015-548/+355
|\ | | | | | | | | Manually resolve conflicts in: rust/migrations/2018-05-12-001226_init/up.sql
| * update TODOBryan Newbold2018-09-101-8/+2
| |
| * many small API cleanupsBryan Newbold2018-09-105-193/+148
| | | | | | | | | | | | - use FatCatId much more often (though not everywhere yet) - more consistent types - remove redundant error handling code in wrappers
| * re-gen rust codeBryan Newbold2018-09-107-61/+133
| |
| * continue CRUD refactor, removing CUD handlersBryan Newbold2018-09-104-276/+60
| | | | | | | | | | | | | | | | Can't remove the "get" handlers because of the expand parameter, which is only implemented for releases and in the handler (not in the CRUD trait, yet). Also didn't do the batch handler stuff yet.
| * squelch unused macro warningBryan Newbold2018-09-101-0/+2
| |
| * database_entity_crud -> api_entity_crudBryan Newbold2018-09-105-5/+5
| |
| * remove 'IS NOT NULL' identifier index constraintsBryan Newbold2018-09-101-14/+14
| | | | | | | | | | | | | | | | | | | | | | These seemed to be resulting in table scans on 404s in QA with postgres 10, despite the adding "IS NOT NULL" WHERE clauses earlier. Query time was very significant, even for the creator table (1.2 seconds or so on SSD). I looked at using hash indices (which have improved in postgres 10), which could save index size (disk and RAM) and potentially be faster for these trivial exact lookups, but didn't go for it at this time.
* | switch entity acceptance for cockroach branchBryan Newbold2018-09-091-10/+10
| |
* | test: contribs should work without wiping databaseBryan Newbold2018-09-091-1/+1
| | | | | | | | | | | | This removes an entanglement between tests that was breaking on the cockroachdb branch (where the database isn't reset between each individual test).
* | changelog ID as a sequenceBryan Newbold2018-09-092-1/+5
| | | | | | | | | | | | | | | | | | | | | | As noted elsewhere, auto-increment sequences in postgres, mysql, and cockroach can have gaps: when transactions that increment a number are rolled back, a "gap" is left. I didn't realize this, and it's problematic for the external changelog API. I'll need to re-write the changelog inserter to query for the current highest number when inserting, which is unfortunate and might slow down bulk imports.
* | Merge branch 'http-verbs' into cockroachBryan Newbold2018-09-093-50/+159
|\|
| * generic edit accept, and per-row variantBryan Newbold2018-09-092-44/+158
| | | | | | | | The per-row variant is for use with cockroach.
| * api_server: give up and use star schema importBryan Newbold2018-09-091-6/+1
| |
* | Merge branch 'http-verbs' into cockroachBryan Newbold2018-09-0922-859/+5239
|\| | | | | | | | | | | | | | | Manually merged conflicts: rust/migrations/2018-05-12-001226_init/up.sql rust/src/api_server.rs rust/src/database_schema.rs
| * update TODOBryan Newbold2018-09-091-9/+11
| |
| * parallelize abstracts insertionBryan Newbold2018-09-091-15/+20
| |
| * TODO and commentBryan Newbold2018-09-072-1/+2
| |
| * fixes after autoaccept mergeBryan Newbold2018-09-074-49/+41
| |
| * re-codegenBryan Newbold2018-09-071-1/+11
| |
| * fmtBryan Newbold2018-09-075-207/+364
| |
| * Merge branch 'autoaccept' into http-verbsBryan Newbold2018-09-0711-63/+475
| |\ | | | | | | | | | | | | | | | | | | | | | | | | Started resolving conflicts in: TODO notes/cloud_instances.txt rust/fatcat-api/README.md rust/src/api_server.rs
| | * editgroup IDs are fcid, not uuidBryan Newbold2018-08-301-1/+1
| | |
| | * minimal autoaccept testsBryan Newbold2018-08-301-0/+41
| | |
| | * editgroup parameter for batch insertsBryan Newbold2018-08-302-6/+21
| | |
| | * WIP: autoacceptBryan Newbold2018-08-248-47/+225
| | |
| | * implement autoacceptBryan Newbold2018-08-202-20/+46
| | | | | | | | | | | | | | | | | | Need to review: - changelog creation - clobbering of existing editgroup flag
| | * codegenBryan Newbold2018-08-208-51/+223
| | |
| * | fix/replace a lot of expect() callsBryan Newbold2018-09-073-47/+48
| | |
| * | mostly done with CRUD refactorBryan Newbold2018-09-073-625/+745
| | | | | | | | | | | | One failing test in this commit.
| * | batch inserts by defaultBryan Newbold2018-09-073-27/+95
| | |
| * | generic create (multiple inserts)Bryan Newbold2018-09-071-26/+24
| | | | | | | | | | | | | | | | | | | | | This does away entirely with the fancy CTE-based custom SQL inserts. Because performance is currently only important for batch inserts, I think this is acceptable, and will refactor to do batch inserts as actual batch SQL operations next.
| * | crud: refactor to split out rev insertBryan Newbold2018-09-071-43/+51
| | |
| * | major CRUD refactorBryan Newbold2018-09-075-85/+336
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the start of a large refactor to move all entity CRUD (create, read, update, delete) model/database code into it's own file. HACKING has been updated with an overview of what happens in each file. Next steps: - split rev (and sub-table) insertion in to db_rev_insert and make create/update generic - inserts should be batch (vector) by default - move all other entities into this new trait framework - bypass api_server wrappers and call into CRUD from api_wrappers for entity ops (should be a big cleanup)
| * | update TODOBryan Newbold2018-09-061-0/+7
| | |
| * | skeleton create/delete endpoints for worksBryan Newbold2018-09-063-2/+293
| | | | | | | | | | | | Other entities just stubs
| * | codegen put/deleteBryan Newbold2018-09-069-42/+3101
| | |
| * | correct EditgroupAlreadyAccepted responseBryan Newbold2018-08-314-5/+14
| | |
| * | add bad request response type for eg acceptBryan Newbold2018-08-317-10/+14
| | |
| * | split/move docs aroundBryan Newbold2018-08-243-84/+61
| | |
| * | NOTES => rust_libraries.txtBryan Newbold2018-08-241-19/+0
| |/
| * use index_val, not index, in SQL schemaBryan Newbold2018-08-204-16/+16
| |
| * add NOT NULL to other lookupsBryan Newbold2018-08-191-0/+6
| |
| * add hints for postgres query plannerBryan Newbold2018-08-191-0/+6
| |
| * add index on release_rev_abstract SHA1 columnBryan Newbold2018-08-191-0/+1
| |
| * add index on file_rev_url file_revBryan Newbold2018-08-191-0/+2
| |
| * create index on release_rev_abstractsBryan Newbold2018-08-181-0/+2
| |
| * ORCIDs can end in X characterBryan Newbold2018-08-164-6/+14
| |
* | cockroach: refactor custom insert queries into dieselBryan Newbold2018-08-171-100/+95
| | | | | | | | | | | | | | | | | | | | | | Basically reverts some big speedups around "single query inserts" (to ident, rev, and edit tables concurrently), because cockroach doesn't support "multiple uses of CTEs" (table expressions). There's probably a way to rewrite using subqueries instead. Also, would actually want to refactor and use NewRow structs instead of tuples, I think.
* | cockroach: catch remaining JSONB stuffBryan Newbold2018-08-172-14/+14
| |