Repository: ray-project/ray
File: python/ray/data/_internal/util.py (Python)

Code before:


def _validate_head_node_resources_for_local_scheduling(
    ray_remote_args: Dict[str, Any],
    *,
    op_name: str,
    required_num_cpus: int = 1,
    required_num_gpus: int = 0,
    required_memory: int = 0,
) -> None:
    """Ensure the head node has enough resources before pinning work there.

    Local paths (``local://``) and other driver-local I/O force tasks onto the
    head node via ``NodeAffinitySchedulingStrategy``. If the head node was
    intentionally started with zero logical resources (a common practice to
    avoid OOMs), those tasks become unschedulable. Detect this upfront and
    raise a clear error with remediation steps.
    """

    # Ray defaults to reserving 1 CPU per task when num_cpus isn't provided.
    num_cpus = ray_remote_args.get("num_cpus", required_num_cpus)
    num_gpus = ray_remote_args.get("num_gpus", required_num_gpus)
    memory = ray_remote_args.get("memory", required_memory)

    required_resources: Dict[str, float] = {}
    if num_cpus > 0:
        required_resources["CPU"] = float(num_cpus)
    if num_gpus > 0:
        required_resources["GPU"] = float(num_gpus)
    if memory > 0:
        required_resources["memory"] = float(memory)

    # Include any additional custom resources requested.
    custom_resources = ray_remote_args.get("resources", {})
    for name, amount in custom_resources.items():
        if amount is None:
            continue
        try:
            amount = float(amount)
        except (TypeError, ValueError) as err:
            raise ValueError(f"Invalid resource amount for '{name}': {amount}") from err
        if amount > 0:
            required_resources[name] = amount

    # If there are no positive resource requirements, the task can run on a
    # zero-resource head node (e.g., num_cpus=0 opt-out), so nothing to check.
    if not required_resources:
        return

    head_node = next(
        (

Review comment (suggestion):
thinking about this more, are these `if num_cpus > 0` checks necessary? I think if they are 0, then we should catch it when building the insufficient dict (since we default to 0)

Code after:


def _truncated_repr(obj: Any) -> str:
    """Utility to return a truncated object representation for error messages."""
    msg = str(obj)
    if len(msg) > 200:
        msg = msg[:200] + "..."
    return msg


def _insert_doc_at_pattern(
    obj,
    *,
    message: str,
    pattern: str,
    insert_after: bool = True,
    directive: Optional[str] = None,
    skip_matches: int = 0,
) -> str:
    if "\n" in message:
        raise ValueError(
            "message shouldn't contain any newlines, since this function will insert "
            f"its own linebreaks when text wrapping: {message}"
        )

    doc = obj.__doc__.strip()
    if not doc:
        doc = ""

    if pattern == "" and insert_after:
        # Empty pattern + insert_after means that we want to append the message to the
        # end of the docstring.
        head = doc
        tail = ""
    else:
        tail = doc
        i = tail.find(pattern)
        skip_matches_left = skip_matches
        while i != -1:
            if insert_after:
                # Set offset to the first character after the pattern.
                offset = i + len(pattern)
            else:
                # Set offset to the first character in the matched line.
                offset = tail[:i].rfind("\n") + 1
            head = tail[:offset]
            tail = tail[offset:]
            skip_matches_left -= 1
            if skip_matches_left <= 0:
                break
            elif not insert_after: