beliefer opened a new pull request, #37013:
URL: https://github.com/apache/spark/pull/37013

   ### What changes were proposed in this pull request?
   https://github.com/apache/spark/pull/36726 supports TimestampNTZ type in 
JDBC data source.
   But the implement is incorrect.
   This PR just modify a test case and it will be failed !
   
   The test case output failure show below.
   ```
   Results do not match for query:
   Timezone: 
sun.util.calendar.ZoneInfo[id="Africa/Dakar",offset=0,dstSavings=0,useDaylight=false,transitions=3,lastRule=null]
   Timezone Env: 
   
   == Parsed Logical Plan ==
   Relation [TIMESTAMP_NTZ '1500-01-20 00:00:00.123456'#253] 
JDBCRelation(timestamp_ntz_diff_tz_support_table) [numPartitions=1]
   
   == Analyzed Logical Plan ==
   TIMESTAMP_NTZ '1500-01-20 00:00:00.123456': timestamp_ntz
   Relation [TIMESTAMP_NTZ '1500-01-20 00:00:00.123456'#253] 
JDBCRelation(timestamp_ntz_diff_tz_support_table) [numPartitions=1]
   
   == Optimized Logical Plan ==
   Relation [TIMESTAMP_NTZ '1500-01-20 00:00:00.123456'#253] 
JDBCRelation(timestamp_ntz_diff_tz_support_table) [numPartitions=1]
   
   == Physical Plan ==
   *(1) Scan JDBCRelation(timestamp_ntz_diff_tz_support_table) 
[numPartitions=1] [TIMESTAMP_NTZ '1500-01-20 00:00:00.123456'#253] 
PushedFilters: [], ReadSchema: struct<TIMESTAMP_NTZ '1500-01-20 
00:00:00.123456':timestamp_ntz>
   
   == Results ==
   
   == Results ==
   !== Correct Answer - 1 ==                                           == Spark 
Answer - 1 ==
    struct<TIMESTAMP_NTZ '1500-01-20 00:00:00.123456':timestamp_ntz>   
struct<TIMESTAMP_NTZ '1500-01-20 00:00:00.123456':timestamp_ntz>
   ![1500-01-20T00:00:00.123456]                                       
[1500-01-20T00:16:08.123456]
       
          
   ScalaTestFailureLocation: org.apache.spark.sql.QueryTest$ at 
(QueryTest.scala:243)
   org.scalatest.exceptions.TestFailedException: 
   ```
   
   ### Why are the changes needed?
   Show an implement bug.
   
   
   ### Does this PR introduce _any_ user-facing change?
   'No'.
   Just show an implement bug.
   
   
   ### How was this patch tested?
   New test case.
   


-- 
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]

Reply via email to