-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Allow spaces in input file paths #9426
Conversation
...-common/src/main/java/org/apache/pinot/common/segment/generation/SegmentGenerationUtils.java
Outdated
Show resolved
Hide resolved
...-common/src/main/java/org/apache/pinot/common/segment/generation/SegmentGenerationUtils.java
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #9426 +/- ##
============================================
+ Coverage 69.77% 69.83% +0.06%
- Complexity 5092 5097 +5
============================================
Files 1890 1890
Lines 100654 100954 +300
Branches 15327 15348 +21
============================================
+ Hits 70231 70506 +275
- Misses 25445 25466 +21
- Partials 4978 4982 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
} | ||
|
||
|
||
public static void main(String[] args) throws Exception { |
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.
You may not need this code change in your PR.
@@ -606,4 +607,26 @@ public void close() | |||
throws IOException { | |||
super.close(); | |||
} | |||
|
|||
public static String sanitizeURIString(String uriStr) { |
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.
Same here. You may not need this code change.
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.
LGTM.
Co-authored-by: Kartik Khare <kharekartik@Kartiks-MacBook-Pro.local>
Replacing spaces with
%20
in segment utils helps out with this process.Not using URLEncoder here since it also encodes the
/
and:
charactersdue to which
uri.getScheme
returnsnull