richardfogaca commented on PR #40100: URL: https://github.com/apache/superset/pull/40100#issuecomment-4477650215
@aminghadersohi I pushed d131f0d1f90c56c3f404a19ec68ec3b07d001002 with a small cleanup for your notes and the automated review suggestions: - Replaced direct dynamic record reads with an own-property helper, without restricting valid Superset column names. - Narrowed the polygon parsing catch to only skip expected parse/shape errors and rethrow unexpected failures. - Left interior-ring/hole handling unchanged since that is pre-existing behavior and better suited for follow-up scope. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
