[
https://issues.apache.org/jira/browse/BEAM-3072?focusedWorklogId=229284&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-229284
]
ASF GitHub Bot logged work on BEAM-3072:
----------------------------------------
Author: ASF GitHub Bot
Created on: 17/Apr/19 18:31
Start Date: 17/Apr/19 18:31
Worklog Time Spent: 10m
Work Description: tvalentyn commented on pull request #8158: [BEAM-3072]
updates to that the error handling and collected the fail…
URL: https://github.com/apache/beam/pull/8158#discussion_r276377657
##########
File path: sdks/python/apache_beam/utils/processes_test.py
##########
@@ -103,5 +104,124 @@ def test_method_forwarding_windows(self, *unused_mocks):
other_arg=True)
+class TestErrorHandlingCheckCall(unittest.TestCase):
+ @classmethod
+ def setup_class(cls):
+ cls.mock_get_patcher = mock.patch(\
+ 'apache_beam.utils.processes.subprocess.check_call')
+ cls.mock_get = cls.mock_get_patcher.start()
+
+ @classmethod
+ def teardown_class(cls):
+ cls.mock_get_patcher.stop()
+
+ def test_oserror_check_call(self):
Review comment:
`setup_class` method is called by nose:
https://github.com/nose-devs/nose/blob/7c26ad1e6b7d308cafa328ad34736d34028c122a/nose/suite.py#L134.
tox suites executes all tests using nose, however when we run individual tests
we run plain `unittest.runner`.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 229284)
Time Spent: 5h 20m (was: 5h 10m)
> Improve error handling at staging time time for DataflowRunner
> --------------------------------------------------------------
>
> Key: BEAM-3072
> URL: https://issues.apache.org/jira/browse/BEAM-3072
> Project: Beam
> Issue Type: Bug
> Components: sdk-py-core
> Reporter: Ahmet Altay
> Assignee: niklas Hansson
> Priority: Minor
> Labels: starter, triaged
> Time Spent: 5h 20m
> Remaining Estimate: 0h
>
> dependency.py calls out to external process to collect dependencies:
> https://github.com/apache/beam/blob/de7cc05cc67d1aa6331cddc17c2e02ed0efbe37d/sdks/python/apache_beam/runners/dataflow/internal/dependency.py#L263
> If these calls fails, the error is not clear. The error only tells what
> failed but does not show the actual error message, and is not helpful for
> users.
> As a general fix processes.py should have general better output collection
> from failed processes.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
