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

2020-04-19 Thread GitBox
maropu commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616086332 retest this please This is an aut

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

2020-04-19 Thread GitBox
maropu commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616087306 I think `createOrReplaceTempView` doesn't throw `TableAlreadyExistsException` without `withTempVie

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

2020-04-19 Thread GitBox
maropu commented on issue #28239: [SPARK-31467][SQL][TEST] Refactor the sql tests to prevent TableAlreadyExistsException URL: https://github.com/apache/spark/pull/28239#issuecomment-616088399 Ur, I see. cc: @dongjoon-hyun Th

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

2020-04-19 Thread GitBox
maropu commented 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? --

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

2020-04-20 Thread GitBox
maropu commented on issue #28239: URL: https://github.com/apache/spark/pull/28239#issuecomment-616365028 Ah, I see. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub