8000 [PREGEL] Split Graph.h, make PregelShard its own struct by markuspf · Pull Request #17963 · arangodb/arangodb · GitHub
[go: up one dir, main page]
Skip to content

[PREGEL] Split Graph.h, make PregelShard its own struct #17963

New issue < 8000 details-dialog class="Box Box--overlay d-flex flex-column anim-fade-in fast overflow-auto" aria-label="Sign up for GitHub">

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Feb 6, 2023
Merged

[PREGEL] Split Graph.h, make PregelShard its own struct #17963

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
d7e1064
Provide test binary for Pregel
Jan 11, 2023
c6711a7
move Graph.h to its own directory
Jan 10, 2023
12de151
Split Graph.h into Vertex.h/VertexID.h/Edge.h
Jan 11, 2023
741f47c
Split out PregelShard, too
Jan 11, 2023
1954b4c
Change header
Jan 26, 2023
e3d55fa
Rename Graph/ to GraphStore/ to address ambiguities
Jan 27, 2023
0ae81ef
Rename Pregel/Graph/ to Pregel/GraphFormat to avoid ambiguities
Jan 27, 2023
6cc8c21
Rename PregelID to VertexID
Jan 17, 2023
4e93ad7
Introduce PregelShard as its own struct
Jan 27, 2023
f2aae42
Fix some complaints of the MSVC compiler
Feb 6, 2023
03ee368
Fix some complaints of the MSVC compiler
Feb 6, 2023
63ac1eb
clang-format
Feb 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix some complaints of the MSVC compiler
  • Loading branch information
Markus Pfeiffer committed Feb 6, 2023
commit 03ee368baa5ff116522a18ac5db53553ab623296
2 changes: 1 addition & 1 deletion arangod/Pregel/Worker/Worker.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -566,7 +566,7 @@ auto Worker<V, E, M>::aqlResult(bool withId) const -> PregelResults {
for (; it.hasMore(); ++it) {
Vertex<V, E> const* vertexEntry = *it;

TRI_ASSERT(vertexEntry->shard() < _config.globalShardIDs().size());
TRI_ASSERT(vertexEntry->shard().value < _config.globalShardIDs().size());
ShardID const& shardId = _config.globalShardID(vertexEntry->shard());

results.openObject(/*unindexed*/ true);
Expand Down
0