-
Notifications
You must be signed in to change notification settings - Fork 504
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
fix mistake multiple backend info as no backend info #723
Conversation
33c1772
to
ee40d99
Compare
Codecov Report
@@ Coverage Diff @@
## master #723 +/- ##
============================================
+ Coverage 71.12% 71.13% +<.01%
- Complexity 4330 4332 +2
============================================
Files 281 281
Lines 20693 20703 +10
Branches 2919 2921 +2
============================================
+ Hits 14718 14727 +9
+ Misses 4527 4526 -1
- Partials 1448 1450 +2
Continue to review full report at Codecov.
|
a5f578c
to
edc3950
Compare
@@ -67,9 +71,19 @@ public void init() { | |||
PropertyKey pkey = null; | |||
try { | |||
pkey = schema.getPropertyKey(PK_BACKEND_INFO); | |||
} catch (BackendException | IllegalStateException | | |||
UncheckedIOException ignored) { | |||
} catch (BackendException | UncheckedIOException ignored) { |
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.
please rebase https://github.com/hugegraph/hugegraph/pull/708
Change-Id: I4f017d7763698c7ccd01eb3b98f0fe897d8448c4
edc3950
to
9b45d0e
Compare
this.graph.name(), | ||
this.graph.backend()); | ||
} | ||
// ignore |
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.
expect throw e here, since we assume the backend is inited before calling info()
Change-Id: I4f017d7763698c7ccd01eb3b98f0fe897d8448c4