[GitHub] [spark] maropu edited a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
maropu edited a comment on issue #28239: URL: https://github.com/apache/spark/pull/28239#issuecomment-616252472 btw, since I think we tend to forget the cleanup, any other solution to automatically clean up temp views with low overheads after each test, if this issue's annoying?

[GitHub] [spark] maropu edited a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException

2020-04-19 Thread GitBox
maropu edited a comment on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-615968152 I think (trivial though), the change looks ok. Did you check them only in `SQLQuerySuite`?