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
[ISSUE #4454] Support File Sink Connector #4455
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4455 +/- ##
============================================
- Coverage 16.95% 15.94% -1.01%
- Complexity 1458 1476 +18
============================================
Files 619 689 +70
Lines 25740 27525 +1785
Branches 2414 2568 +154
============================================
+ Hits 4365 4390 +25
- Misses 20936 22686 +1750
- Partials 439 449 +10
... and 8 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@Override | ||
public void start() throws Exception { | ||
if (fileName == null || fileName.length() == 0 || filePath == null || filePath.length() == 0) { | ||
this.outputStream = System.out; | ||
} else { | ||
this.outputStream = | ||
new PrintStream(Files.newOutputStream(Paths.get(filePath + fileName), StandardOpenOption.CREATE, StandardOpenOption.APPEND), | ||
false, | ||
StandardCharsets.UTF_8.name()); | ||
} | ||
} |
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.
Is it better to have a extra line break between L93 and L94?
Fixes #4454 .
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Describe the modifications you've done.
Documentation