-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-7405][Streaming] Fix the bug that ReceiverInputDStream doesn't report InputInfo #5950
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
Conversation
The bug is because SPARK-7139 removed some codes from SPARK-7112 unintentionally here: 1854ac3#diff-5c8651dd78abd20439b8eb938175075dL72
|
Merged build triggered. |
|
Merged build started. |
|
Test build #32023 has started for PR 5950 at commit |
|
Test build #32023 has finished for PR 5950 at commit
|
|
Merged build finished. Test FAILed. |
|
Test FAILed. |
|
retest this please. |
|
Merged build triggered. |
|
Merged build started. |
|
Test build #32043 has started for PR 5950 at commit |
|
That was super-unfortunate. And I missed it as well. It shows that we didnt have sufficient test coverage on that. LGTM, will merge when tests pass. |
|
Test build #32043 has finished for PR 5950 at commit
|
|
Merged build finished. Test PASSed. |
|
Test PASSed. |
|
Merging. |
…t report InputInfo The bug is because SPARK-7139 removed some codes from SPARK-7112 unintentionally here: 1854ac3#diff-5c8651dd78abd20439b8eb938175075dL72 This PR just added them back and added some assertions in the tests to verify it. Author: zsxwing <[email protected]> Closes #5950 from zsxwing/SPARK-7405 and squashes the following commits: 675f5d9 [zsxwing] Fix the bug that ReceiverInputDStream doesn't report InputInfo (cherry picked from commit 14502d5) Signed-off-by: Tathagata Das <[email protected]>
…t report InputInfo The bug is because SPARK-7139 removed some codes from SPARK-7112 unintentionally here: apache@1854ac3#diff-5c8651dd78abd20439b8eb938175075dL72 This PR just added them back and added some assertions in the tests to verify it. Author: zsxwing <[email protected]> Closes apache#5950 from zsxwing/SPARK-7405 and squashes the following commits: 675f5d9 [zsxwing] Fix the bug that ReceiverInputDStream doesn't report InputInfo
…t report InputInfo The bug is because SPARK-7139 removed some codes from SPARK-7112 unintentionally here: apache@1854ac3#diff-5c8651dd78abd20439b8eb938175075dL72 This PR just added them back and added some assertions in the tests to verify it. Author: zsxwing <[email protected]> Closes apache#5950 from zsxwing/SPARK-7405 and squashes the following commits: 675f5d9 [zsxwing] Fix the bug that ReceiverInputDStream doesn't report InputInfo
…t report InputInfo The bug is because SPARK-7139 removed some codes from SPARK-7112 unintentionally here: apache@1854ac3#diff-5c8651dd78abd20439b8eb938175075dL72 This PR just added them back and added some assertions in the tests to verify it. Author: zsxwing <[email protected]> Closes apache#5950 from zsxwing/SPARK-7405 and squashes the following commits: 675f5d9 [zsxwing] Fix the bug that ReceiverInputDStream doesn't report InputInfo
The bug is because SPARK-7139 removed some codes from SPARK-7112 unintentionally here: 1854ac3#diff-5c8651dd78abd20439b8eb938175075dL72
This PR just added them back and added some assertions in the tests to verify it.