Skip to content

[COR] Unify Provider Steps #22144

New issue

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

Closed
markuspf wants to merge 9 commits into from
Closed

[COR] Unify Provider Steps #22144

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
bf89601
Remove cursorID from Steps
markuspf Nov 27, 2025
9a74fbf
Use type alias and remove unused function
markuspf Nov 28, 2025
12fa0a1
Simplify Step code
markuspf Nov 28, 2025
17c8d09
Clean up includes
markuspf Nov 28, 2025
e4abe36
Simplify Step code
markuspf Nov 28, 2025
f8f0104
Simplify Step code
markuspf Nov 28, 2025
2d53f04
Simplify Step code even more
markuspf Nov 28, 2025
d7e1d2c
Fix tests
markuspf Nov 28, 2025
b21a045
restore isValid
markuspf Nov 28, 2025
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
Simplify Step code
  • Loading branch information
@markuspf
markuspf committed Nov 28, 2025
commit e4abe36d6e0ce6cffea122e7a051d48637f5a2d4
4 changes: 2 additions & 2 deletions arangod/Graph/Providers/ClusterProvider.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -500,9 +500,9 @@ auto ClusterProvider<StepImpl>::fetchVertices(
// in that case we do not have to fetch the actual vertex data

for (auto& lE : looseEnds) {
if (!_opts.getCache()->isVertexCached(lE->getVertexIdentifier())) {
if (!_opts.getCache()->isVertexCached(lE->getVertex().getID())) {
// we'll only cache the vertex id, we do not need the data
_opts.getCache()->cacheVertex(lE->getVertexIdentifier(),
_opts.getCache()->cacheVertex(lE->getVertex().getID(),
VPackSlice::nullSlice());
}
result.emplace_back(lE);
Expand Down
5 changes: 0 additions & 5 deletions arangod/Graph/Steps/ClusterProviderStep.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,11 +121,6 @@ class ClusterProviderStep : public arangodb::graph::BaseStep {
}
bool isUnknown() const noexcept { return _validationStatus.isUnknown(); }

// beware: returns a *copy* of the vertex id
[[nodiscard]] VertexType getVertexIdentifier() const {
return _vertex.getID();
}

// beware: returns a *copy* of the edge id
[[nodiscard]] EdgeType getEdgeIdentifier() const { return _edge.getID(); }

Expand Down
3 changes: 0 additions & 3 deletions arangod/Graph/Steps/SingleServerProviderStep.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,6 @@ class SingleServerProviderStep : public arangodb::graph::BaseStep {
bool isLooseEnd() const { return false; }
bool isUnknown() const { return false; }

// beware: will return a *copy* of the vertex id
VertexType getVertexIdentifier() const { return _vertex.getID(); }

// beware: will return a *copy* of the edge id
EdgeType getEdgeIdentifier() const { return _edge.getID(); }

Expand Down