sumeetgajjar commented on pull request #34693:
URL: https://github.com/apache/spark/pull/34693#issuecomment-976972982


   > This change also seem to work with MSSQL's temp table syntax:
   > 
   > ```
   > val withClause = "(SELECT * INTO #TempTable FROM (SELECT * FROM tbl WHERE 
x > 10) t)"
   > val query = "SELECT * FROM #TempTable"
   > val df = spark.read.format("jdbc")
   >   .option("url", jdbcUrl)
   >   .option("withClause", withClause)
   >   .option("query", query)
   >   .load()
   > ```
   
   Since it also works with temp table syntax, do you think it would be a good 
idea to include it the title of this PR and modify the title to "Support CTE 
and TempTable queries with MSSQL JDBC"?


-- 
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-unsubscr...@spark.apache.org

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



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

Reply via email to