Skip to content

[C++][IPC] Consider validation of index sizes in SparseCSFIndex::Make #50308

Description

@tadeja

Describe the enhancement requested

Currently SparseCSFIndex::Make (cpp/src/arrow/sparse_tensor.cc) builds the index tensors from indices_shapes without validation:

    indptr[i] = std::make_shared<Tensor>(indptr_type, indptr_data[i],
                                         std::vector<int64_t>({indices_shapes[i] + 1}));
...
    indices[i] = std::make_shared<Tensor>(indices_type, indices_data[i],
                                          std::vector<int64_t>({indices_shapes[i]}));

Review what validation and place would be best.

Component(s)

C++

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions