Kimahriman commented on PR #711:
URL: https://github.com/apache/incubator-sedona/pull/711#issuecomment-1321191422

   Looks good to me. Only other comment would be if the tests could be 
parameterized at all so to dedupe some of it, and maybe verify the schema and 
at least one row of data to make sure the output is correct and we don't run 
into a similar problem as https://github.com/apache/incubator-sedona/pull/706


-- 
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: dev-unsubscr...@sedona.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to