You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "valentinp17 (via GitHub)" <gi...@apache.org> on 2023/09/14 22:27:08 UTC

[GitHub] [spark] valentinp17 commented on a diff in pull request #42884: [SPARK-42304][FOLLOWUP][SQL] Add test for `GET_TABLES_BY_TYPE_UNSUPPORTED_BY_HIVE_VERSION`

valentinp17 commented on code in PR #42884:
URL: https://github.com/apache/spark/pull/42884#discussion_r1326575472


##########
.github/PULL_REQUEST_TEMPLATE:
##########
@@ -9,7 +9,7 @@ Thanks for sending a pull request!  Here are some tips for you:
   7. If you want to add a new configuration, please read the guideline first for naming configurations in
      'core/src/main/scala/org/apache/spark/internal/config/ConfigEntry.scala'.
   8. If you want to add or modify an error type or message, please read the guideline first in
-     'core/src/main/resources/error/README.md'.
+     'common/utils/src/main/resources/error/README.md'.

Review Comment:
   Agree, I will create a new Jira ticket and PR for it



-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org