New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 848329 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Investigate whether pipeline input is needed for the OnTimeout method in pipelines.py

Project Member Reported by liaoyuke@chromium.org, May 31 2018

Issue description

def OnTimeout(self, arg, parameters):  # pylint: disable=unused-argument
    """This function is called upon timeout without timely callback.

    Args:
      arg (input_type): The input of the pipeline.
      parameters (dict): A mapping from names to string values of the saved
          parameters.
    """
    logging.error('%s.OnTimeout called with %s after %s seconds',
                  type(self).__name__, parameters, self.TimeoutSeconds())

this method introduces arg as input, but never used it, thus causing pylint errors.
 

Comment 1 by st...@chromium.org, Jun 1 2018

Status: WontFix (was: Untriaged)
That is for the baseclass and is needed for subclasses. Let's still keep it.

Sign in to add a comment