-
Notifications
You must be signed in to change notification settings - Fork 32
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
Exchange network backend from Gsocket to libfabric #124
base: master
Are you sure you want to change the base?
Changes from all commits
7884e48
faced31
0d12075
0527b66
2516f77
340cce7
a619897
4793355
5d50465
ed03ab4
d4f0673
d04b2b0
1968022
8666f0f
859f91e
578fc2b
20a7e00
16b693b
279418e
f31b35e
9cd6dd0
71d6438
304b596
48f590d
dff90ef
dfb3ca2
f0db28d
9eb831f
039d313
13c6117
f63e567
de255ca
0003fd8
90c4f43
e99a8fd
65a31eb
108f2a2
918b94b
73f0c85
561e313
c3a608c
e4e3857
639b53c
9e570fb
e0d6927
925207f
0515521
833fa3d
2ae6e61
f6ea573
d51d50e
93c0a95
99d136c
a15b56b
919ff5c
770927e
cb7d9ad
830a888
82ea7f3
9c036dd
4d5e8bb
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -109,7 +109,7 @@ jobs: | |
sudo apt --yes --purge autoremove | ||
sudo aa-remove-unknown | ||
sudo apt update || true | ||
sudo apt --yes --no-install-recommends install ninja-build pkgconf libglib2.0-dev libbson-dev liblmdb-dev libsqlite3-dev libleveldb-dev libmongoc-dev libmariadb-dev librocksdb-dev libfuse3-dev libopen-trace-format-dev librados-dev | ||
sudo apt --yes --no-install-recommends install ninja-build pkgconf libglib2.0-dev libbson-dev liblmdb-dev libsqlite3-dev libleveldb-dev libmongoc-dev libmariadb-dev librocksdb-dev libfuse3-dev libopen-trace-format-dev librados-dev libfabric-dev | ||
sudo apt --yes --no-install-recommends install python3 python3-pip python3-setuptools python3-wheel | ||
sudo pip3 install meson | ||
- name: Cache dependencies | ||
|
@@ -198,6 +198,10 @@ jobs: | |
- os: | ||
dist: ubuntu-18.04 | ||
dependencies: system | ||
# libfaibrc don't work on Ubuntu 20.04 | ||
- os: | ||
dist: ubuntu-20.04 | ||
dependencies: system | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Disable system test because libfabric won't start properly |
||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v2 | ||
|
@@ -211,7 +215,7 @@ jobs: | |
sudo apt --yes --purge autoremove | ||
sudo aa-remove-unknown | ||
sudo apt update || true | ||
sudo apt --yes --no-install-recommends install ninja-build pkgconf libglib2.0-dev libbson-dev liblmdb-dev libsqlite3-dev libleveldb-dev libmongoc-dev libmariadb-dev librocksdb-dev libfuse3-dev libopen-trace-format-dev librados-dev | ||
sudo apt --yes --no-install-recommends install ninja-build pkgconf libglib2.0-dev libbson-dev liblmdb-dev libsqlite3-dev libleveldb-dev libmongoc-dev libmariadb-dev librocksdb-dev libfuse3-dev libopen-trace-format-dev librados-dev libfabric-dev | ||
sudo apt --yes --no-install-recommends install python3 python3-pip python3-setuptools python3-wheel | ||
sudo pip3 install meson | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add libfabric to dependencies |
||
- name: Cache dependencies | ||
|
@@ -294,9 +298,9 @@ jobs: | |
run: | | ||
. scripts/environment.sh | ||
./scripts/setup.sh start | ||
./scripts/test.sh | ||
./scripts/test.sh || (journalctl GLIB_DOMAIN=JULEA && $(exit 1)) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Print journal from SERVER if test fails |
||
sleep 10 | ||
./scripts/test.sh | ||
./scripts/test.sh || (journalctl GLIB_DOMAIN=JULEA && $(exit 1)) | ||
./scripts/setup.sh stop | ||
- name: HDF5 Tests | ||
if: matrix.julea.db != 'memory' | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1109,7 +1109,7 @@ _backend_schema_get(gpointer backend_data, gpointer _batch, gchar const* name, b | |
return TRUE; | ||
|
||
_error: | ||
if (G_UNLIKELY(!j_sql_reset(thread_variables->sql_backend, prepared->stmt, NULL))) | ||
if (G_UNLIKELY(!thread_variables || !prepared || !j_sql_reset(thread_variables->sql_backend, prepared->stmt, NULL))) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
{ | ||
goto _error2; | ||
} | ||
|
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.
Add libfabric to dependencies