-
Notifications
You must be signed in to change notification settings - Fork 11
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
TigerGraph implementation #37
base: main
Are you sure you want to change the base?
Conversation
```bash | ||
cd ~ | ||
git clone https://github.com/ldbc/ldbc_finbench_transaction_impls.git | ||
cd ~/ldbc_finbench_transaction_impls |
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.
should cd to tigergraph directory
workload=org.ldbcouncil.finbench.driver.workloads.transaction.LdbcFinBenchTransactionWorkload | ||
db=org.ldbcouncil.finbench.impls.tigergraph.TigerGraphTransactionDb | ||
operation_count=1000000 | ||
validation_parameters_size=1000 |
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.
keep the parameters same here
ldbc.finbench.transaction.queries.scale_factor=1 | ||
# Frequency of complex read queries | ||
ldbc.finbench.transaction.queries.ComplexRead1_freq=10 | ||
ldbc.finbench.transaction.queries.ComplexRead2_freq=45 |
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.
keep the parameters same here
|
||
# multi-edge | ||
CREATE DIRECTED EDGE transfer(From Account, To Account, DISCRIMINATOR(timestamp UINT), orderNumber STRING, amount DOUBLE, comment STRING, payType STRING, goodsType STRING) WITH REVERSE_EDGE="transfer_REVERSE" | ||
CREATE DIRECTED EDGE deposit(From Loan, To Account, DISCRIMINATOR(timestamp UINT), amount DOUBLE) WITH REVERSE_EDGE="deposit_REVERSE" |
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.
the keyword for edge multiplicity is DISCRIMINATOR?
gsql -g ldbc_fin "install query -single all" | ||
else | ||
gsql -g ldbc_fin "install query -single trw2" | ||
gsql -g ldbc_fin "set single_gpr=false install query all" |
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.
what is single_gpr used for?
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.
And why is trw2 special?
tigergraph/scripts/ExprFunctions.hpp
Outdated
|
||
using namespace std; | ||
|
||
inline uint64_t convert_time_to_uint(string str_time) { |
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.
convert_time_to_uint
function is not used
Nodes = SELECT s FROM Nodes:s | ||
POST-ACCUM | ||
s.isBlocked = (s.@outCount > 0 AND s.@totalE1Amount/s.@totalE2Amount <= ratioThreshold) OR (s.@outCount == 0 AND ratioThreshold >= -1) | ||
; |
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.
no rollback here?
Implementing queries through GSQL's v1 syntax.