-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-3586][streaming]Support nested directories in Spark Streaming #2765
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
Closed
Closed
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
4a7eef4
Support nested directories in Spark Streaming
wangxiaojing 50ad7d4
change Nit
wangxiaojing c14def1
support depth
wangxiaojing bfbec51
Change space before brace
wangxiaojing 0a8ecf8
change process any files created in nested directories
wangxiaojing 1ce623d
reformat code
wangxiaojing fe6e5ca
add a require(depth >= 0)
wangxiaojing 7031940
reformat code
wangxiaojing 7bd4811
change performance
wangxiaojing 05b5fba
change filter name
wangxiaojing e66b166
change line exceeds 100 columns
wangxiaojing a63c5a3
line over 100
wangxiaojing 0b4812e
remove line
wangxiaojing 8990c35
style
wangxiaojing a20743f
change get depth
wangxiaojing d7f4880
Use 'isDir' to modify the compatibility
wangxiaojing 99b05d6
rebase
wangxiaojing b6788a3
support java Api
wangxiaojing d2f606c
Add support python api
wangxiaojing 8e3a054
Additional excludes for checking of Spark's binary compatibility
wangxiaojing 571730a
rebase
wangxiaojing a4bfac2
rebase
wangxiaojing 5e3fd3d
change API
wangxiaojing e4b9c22
change MiMa failures
wangxiaojing 1a2aae9
rebase
wangxiaojing File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This looks out of order, but, I question the need for this overload anyway. Maybe @tdas has an opinion. I'd like to keep the scope of this change much more limited to adding a depth param to 1 version of each method (maybe 2 if it really made sense) in each language.