-
Notifications
You must be signed in to change notification settings - Fork 30
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.
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
MergeV, MergeE, & Option Steps #214
Open
criminosis
wants to merge
52
commits into
wolf4ood:master
Choose a base branch
from
criminosis:new_steps
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
e2c85cf
First pass
criminosis 8486503
Added JanusGraph in memory mode to docker compose enviornment
criminosis e06fe55
Added mergeV as a start step
criminosis b569530
Relaxed property step keys to Into<GValue> from &str
criminosis 5ec7c71
Added JanusGraph custom vertex id tests
criminosis 7cf15cd
Implemented From<HashMap<GKey, GValue>> for MergeVertexStep to suppor…
criminosis 6a0f4c5
Implemented Option step for mergeV
criminosis 0f4aee6
Added healthcheck for JG & wait with timeout for docker compose up in…
criminosis dddf8c0
Use Docker Compose v2 via "docker compose" vs v1's "docker-compose" i…
criminosis e1d7096
Combine merge v custom id test cases
criminosis 3d3a300
Better handle tests being reran
criminosis 6f56392
Added merge_v_tests feature for tests
criminosis dada863
Formatting
criminosis bc94862
FIxed if condition
criminosis 80357fe
Corrected cargo.toml merge_test feature
criminosis 18051d3
Changed GH Action if statement formatting
criminosis 6737d91
Increased docker compose timeout time for healthy service check
criminosis 7a8a0cd
Fixed imports for merge_test module. Moved merge tests into their own…
criminosis 097fa04
Use drop vertices test utility function
criminosis aff1d27
Drop vertices for test_merge_v_options
criminosis 44cdfd3
Added mergeV step to anonymous traversals
criminosis 1c97441
Implemented mergeE step
criminosis 8b28edc
Added mergeV and mergeE to Bytecode WRITE_OPERATORS
criminosis 72615b9
Implemented travsal test based on reference doc combo mergeV and mergeE
criminosis edffffb
Support literal options for choose step and added test
criminosis 7a255df
Rewrote side effect and expose via GraphTraversal
criminosis 09d562c
Implemented support for Columns in By Step
criminosis 97b677c
Expose properties() step in an anonymouse traversal
criminosis f0115d2
Also update property_many, property_with_cardinality, and property_ma…
criminosis f8f5fec
If a request responds with a websocket error mark the conneciton as i…
criminosis 668c564
Map additional tungstenite errors to GremlinError::WebSocketAsync
criminosis cdcc212
Arc tungstenite::Error into WebSocketAsync error to maintain Async er…
criminosis 2b72a1a
Expose healthcheck interval setting on async connection pool
criminosis 093c04c
Formatting
criminosis 802a20c
Map mobc pool errors to type that would invalidate connection
criminosis 03c24fe
Exploratory logging
criminosis a342d8c
Added uuid to connection instance logging
criminosis ec1437d
Update mobc and make its idle connection behavior the same as the rd2…
criminosis 784ca87
0.8 mobc does not treat async-std feature as mutually exclusive from …
criminosis c10ceaa
Include tokio/sync for mobc compilation in async-std-runtime feature
criminosis 6a94a88
Implemented None step
criminosis bce3737
Implemented iterator() method on returned remote stream to consume st…
criminosis 5db2b58
Trial connection multiplexing for non-credential configured clients
criminosis 8082d3b
Formatting
criminosis 7500820
Removed internal channel bounding
criminosis 839ef5e
Revert "Removed internal channel bounding"
criminosis 41c753d
Revert "Formatting"
criminosis 19c9a40
Revert "Trial connection multiplexing for non-credential configured c…
criminosis cebe99a
Revert "Added uuid to connection instance logging"
criminosis 3b894b4
Revert "Exploratory logging"
criminosis f2d4346
Formatting
criminosis 2c5bf51
Switched command to docker compose in coverage GH Action Workflow
criminosis File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file was deleted.
Oops, something went wrong.
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems
docker-compose
(note the middle-
is no longer a known command on the latest action runners, so changed it here and in the test workflow files