zhangfengcdt commented on PR #1525: URL: https://github.com/apache/sedona/pull/1525#issuecomment-2243473487
> > @Kontinuation @zhangfengcdt Can we also emit the values of non-spatial fields to provide more hints? > > I don't think it is possible given the current approach to solve this problem. > > Another concern is: sometimes the exceptions is not raised by ourselves, but somewhere inside JTS. This mostly happens in overlay functions (intersection, union, etc) and some topology-preserving transformation functions. We have to catch exceptions thrown by JTS and convert them to our own `IllegalGeometryException` exception, this is very tedious and certainly not what we want to do. Thanks, @Kontinuation @jiayuasu for the suggestions. I have refactor the code to add the input row string to the exceptions. Also, in case input is null (e.g., rows are created in test using SQL literals), I add the literals to the exception. This will help with the debug cases as well. -- 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-unsubscr...@sedona.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
