mrkn commented on pull request #7477: URL: https://github.com/apache/arrow/pull/7477#issuecomment-650148002
> > Without the canonical flag, we need to make a copy and sort the data of non-canonical sparse tensor when serializing it because the current SparseCOOIndex has the constraint that the indices matrix is sorted. > > Sory, but I don't understand this sentence. If SparseCOOIndex has the constraint that the indices matrix is sorted, then it is canonical already. Why do you need to add a flag? Oh, I missed to explain that this pull request removes that constraint. ---------------------------------------------------------------- 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: us...@infra.apache.org