Support graph read mode and set/list aggregate property by zhoney · Pull Request #1332 · apache/hugegraph
Change-Id: Ic2e651970193b16e7e94fe2c73e58392079a2af5
Change-Id: Ic8d18c111d016cb12ee0fa62a847205b72c7a5b7
Change-Id: Id0b3c025a93112f6dfaaa0e0f886ba9a35a40fc8
Change-Id: I1e1f906b76c5fdba65cb16b7269697d3100ae054
Change-Id: I18d999df2a7dd64f4e95926ca06d1caabd28193e
| public HugeGraph graph(); | ||
| public String name(); | ||
| public GraphMode mode(); | ||
| public GraphReadMode graphReadMode(); |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
readMode
| private DataType dataType; | ||
| private Cardinality cardinality; | ||
| private AggregateType aggregateType; | ||
| private boolean olap; |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prefer define enum ReadFrequency include OLTP and OLAP, maybe we'll add OLTP_COLD in the future
| Assert.assertEquals(120L, edge.value("times")); | ||
| Assert.assertEquals(ImmutableSet.of(21, 22, 23), | ||
| edge.value("port")); | ||
| Assert.assertEquals(ImmutableList.of(21, 22, 23, 23), |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
calcList effected?
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
Change-Id: I5db5a1bc91c14b659bbba27c2ad15b40245bc62a
Linary
deleted the
read-mode-and-set-list-agg
branch
javeme
changed the title
Support graph read mode and set list aggregate property
Support graph read mode and set/list aggregate property
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters