Skip to content
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

Jump to bottom

[master < T0751-MG] Support multiple types for weighted shortest path #278

Merged
merged 7 commits into from
Oct 19, 2021
Merged

[master < T0751-MG] Support multiple types for weighted shortest path #278

merged 7 commits into from
Oct 19, 2021

Conversation

antonio2368
Copy link
Contributor

@antonio2368 antonio2368 commented Oct 18, 2021 •
edited
Loading

[master < Task] PR

  • Check, and update documentation if necessary
  • Update changelog

@antonio2368 antonio2368 self-assigned this Oct 18, 2021
@antonio2368 antonio2368 changed the title Support multiple types for weighted shortest path [master < T0751-MG] Support multiple types for weighted shortest path Oct 18, 2021
@antonio2368 antonio2368 linked an issue Oct 18, 2021 that may be closed by this pull request
[BUG] wShortest doesn't work with temporal types #277
Closed
antaljanosbenjamin
antaljanosbenjamin reviewed Oct 18, 2021
View reviewed changes
src/query/plan/operator.cpp Outdated Show resolved Hide resolved
src/query/plan/operator.cpp Outdated Show resolved Hide resolved
@antonio2368 antonio2368 force-pushed the T0751-MG-fix-duration-shortest-path branch from e030efa to 2b16a2d Compare October 18, 2021 12:10
@antonio2368 antonio2368 marked this pull request as ready for review October 18, 2021 12:12
@antonio2368 antonio2368 merged commit 1a78c36 into master Oct 19, 2021
@antonio2368 antonio2368 deleted the T0751-MG-fix-duration-shortest-path branch October 19, 2021 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gitbuda gitbuda gitbuda approved these changes

@antaljanosbenjamin antaljanosbenjamin antaljanosbenjamin approved these changes

@jbajic jbajic jbajic approved these changes

@kostasrim kostasrim Awaiting requested review from kostasrim

Assignees

@antonio2368 antonio2368

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] wShortest doesn't work with temporal types
5 participants
@antonio2368 @gitbuda @antaljanosbenjamin @kostasrim @jbajic