Skip to content

Conversation

@DvirDukhan
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Jun 6, 2019

Codecov Report

Merging #39 into master will increase coverage by 0.32%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage      68%   68.32%   +0.32%     
==========================================
  Files          16       16              
  Lines         400      401       +1     
  Branches       68       68              
==========================================
+ Hits          272      274       +2     
+ Misses        103      102       -1     
  Partials       25       25
Impacted Files Coverage Δ
...main/java/com/redislabs/redisgraph/RedisGraph.java 89.36% <ø> (ø) ⬆️
.../com/redislabs/redisgraph/graph_entities/Node.java 40% <ø> (ø)
...bs/redisgraph/impl/graph_cache/GraphCacheList.java 100% <ø> (ø)
...a/com/redislabs/redisgraph/impl/ResultSetImpl.java 81.52% <ø> (ø) ⬆️
.../redislabs/redisgraph/graph_entities/Property.java 53.57% <ø> (ø)
...islabs/redisgraph/impl/graph_cache/GraphCache.java 100% <ø> (ø)
.../com/redislabs/redisgraph/graph_entities/Edge.java 32% <ø> (ø)
...dislabs/redisgraph/graph_entities/GraphEntity.java 44% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4eadb76...45399e3. Read the comment docs.

import com.redislabs.redisgraph.impl.Edge;
import com.redislabs.redisgraph.impl.Node;
import com.redislabs.redisgraph.impl.Property;
import com.redislabs.redisgraph.graph_entities.Edge;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why graph_entities? and not graphEntities?

package com.redislabs.redisgraph;

import com.redislabs.redisgraph.impl.GraphCache;
import com.redislabs.redisgraph.impl.graph_cache.GraphCache;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

graph_cache or graphCache

@gkorland gkorland changed the title Update README.md Move graph entities Jun 12, 2019
Copy link
Contributor

@gkorland gkorland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use Camel case for package names

@DvirDukhan DvirDukhan merged commit 6eda1d3 into master Jun 12, 2019
@gkorland gkorland deleted the new-resultset-structure branch June 12, 2019 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants