kensho-technologies / graphql-compiler / 964
89%

DEFAULT BRANCH: master
Ran
Jobs 3
Files 38
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
964

push

travis-ci

obi1kenobi 
Amend metadata table for "explain" support. (#141)

* Amend metadata table for "explain" support.

* QueryMetadataTable.get_explain_infos() returns information required for explain analysis.
It is in the form of a list of ExplainInfo objects.

* ExplainFilterInfo provides:
  - op_name: Operation is useful to estimate complexity, i.e. == is faster than between.
  - field_name: For checking if the field has index or not.
  - args: If there are $args, we can check the size of user provided input.

* ExplainRecurseInfo provides the recursion depth for now.

* fix ExplainFilterInfo -> FilterInfo.

* fix lint error

* remove unused import

* lint issue

* sort imports

* Cleanup explain infos.

- metadata table now has separate calls for FilterInfo and RecurseInfo
- Unittest checks if any extra infos appear.
- Unittest is deriving from the class and using self.assertEqual
- setdefault is used instead of defaultdict

* fix unused import

* Allow multiple RecurseInfos.

We only keep vertices in the metadata table. Multiple recursions at
the same vertex for different edges should be recorded together.
Name and direction of each edge is also added to the RecurseInfo.

* unit test cleanup

- test method names reflect test query name from test_input_data
- has_edge_degree test added
- multiple filters on same field test added
- test expectations are simplified with variables

* fix lint spacing

* review fixes

- FilterInfo field_name renamed to fields and is now a tuple. Most operators have one field, but name_or_alias sets (name,alias) tuple here. This makes it more consistent for callers to process filter's fields.
- FilterInfo args field is now a tuple instead of list.
- assertEqual order is fixed to better error display in tests.
- One more test case for name_or_alias
- Test expectation indentations improved.

* fix lint errors

* Simplify statement.

* fix indent

3141 of 3297 relevant lines covered (95.27%)

1.9 hits per line

Jobs
ID Job ID Ran Files Coverage
1 964.1 0
Travis Job 964.1
2 964.2 0
95.06
 Travis Job 964.2
3 964.3 0
95.27
 Travis Job 964.3
Source Files on build 964
Detailed source file information is not available for this build.