You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "kevingurney (via GitHub)" <gi...@apache.org> on 2023/05/25 17:54:33 UTC
[GitHub] [arrow] kevingurney opened a new issue, #35768: [MATLAB] Enable better support for exception handling in the MATLAB interface
kevingurney opened a new issue, #35768:
URL: https://github.com/apache/arrow/issues/35768
### Describe the enhancement requested
Right now, there is limited support for exception handling in the Proxy classes in the MATLAB interface. To make it easier to handle exceptions, we would need to leverage additional infrastructure in mathworks/libmexclass. Once that infrastructure has been added, we should update the relevant exception throwing call sites in the MATLAB interface.
See https://github.com/apache/arrow/pull/35655#discussion_r1204879669 for an example of this.
### Component(s)
MATLAB
--
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.
To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] sgilmore10 commented on issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "sgilmore10 (via GitHub)" <gi...@apache.org>.
sgilmore10 commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1584489708
Hi @kou,
Closing this as a duplicate issue sounds good to me! I'm not quite sure how to do that, or if I have the right permissions to.
Best,
Sarah
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kou commented on issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "kou (via GitHub)" <gi...@apache.org>.
kou commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1583352268
Can we close this as a duplicated issue?
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kevingurney commented on issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "kevingurney (via GitHub)" <gi...@apache.org>.
kevingurney commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1564535418
Thanks for your input, @kou!
We actually had the same thought process. We would prefer to use an approach analogous to `arrow::Result` / `arrow::Status` rather than relying on C++'s exception handling patterns in the MATLAB interface.
I've updated the title and description of this issue to reflect the fact that we intend to focus primarily on our error handling approach in the MATLAB interface, rather than specifically using C++ exceptions.
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] sgilmore10 commented on issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "sgilmore10 (via GitHub)" <gi...@apache.org>.
sgilmore10 commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1582597383
take
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] sgilmore10 commented on issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "sgilmore10 (via GitHub)" <gi...@apache.org>.
sgilmore10 commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1584486860
Duplicate of #35918
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] sgilmore10 commented on issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "sgilmore10 (via GitHub)" <gi...@apache.org>.
sgilmore10 commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1582598930
This issue was closed by #35914. I made an issue for adding error handling support without realizing this one already existed.
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kevingurney commented on issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "kevingurney (via GitHub)" <gi...@apache.org>.
kevingurney commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1585423597
Thank you, @kou! My apologies - I didn't realize I had the permissions to close the issue myself.
Thanks for letting me know for the future!
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kou commented on issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "kou (via GitHub)" <gi...@apache.org>.
kou commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1585147095
The issue author or maintainers can close an issue.
I close this as duplicate of #35918.
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kou closed issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
Posted by "kou (via GitHub)" <gi...@apache.org>.
kou closed issue #35768: [MATLAB] Enable better support for error handling in the MATLAB interface
URL: https://github.com/apache/arrow/issues/35768
--
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.
To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kou commented on issue #35768: [MATLAB] Enable better support for exception handling in the MATLAB interface
Posted by "kou (via GitHub)" <gi...@apache.org>.
kou commented on issue #35768:
URL: https://github.com/apache/arrow/issues/35768#issuecomment-1563516134
Note that we don't need to use the C++'s exception feature for this. For example, Apache Arrow C++ uses `arrow::Status` instead of the C++'s exception feature.
We just need to propagate an error in Apache Arrow C++ or Apache Arrow MATLAB to an exception in MATLAB.
--
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.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org