-
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 leak of absolute path in gremlin #871
Conversation
Change-Id: I2e066014a3993862f3bea830dd4a683d2c7b6f63
Codecov Report
@@ Coverage Diff @@
## master #871 +/- ##
============================================
- Coverage 72.12% 72.11% -0.01%
Complexity 4450 4450
============================================
Files 287 287
Lines 21201 21208 +7
Branches 2992 2992
============================================
+ Hits 15291 15294 +3
- Misses 4404 4408 +4
Partials 1506 1506
Continue to review full report at Codecov.
|
Change-Id: I31bfc7faa89c226cb26fd4f2154d226904aec0ab
SerializerProvider provider) | ||
throws IOException { | ||
jsonGenerator.writeStartObject(); | ||
jsonGenerator.writeStringField("fileName", file.getName()); |
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.
fileName => file_name or file
Change-Id: I2e066014a3993862f3bea830dd4a683d2c7b6f63