Remove unnecessary condition else check#21423
Conversation
ab7ae19 to
b80863a
Compare
|
2023-11-21T00:20:32.223-0600 WARN TestNG-test=Surefire test-1 org.eclipse.jetty.util.thread.QueuedThreadPool QueuedThreadPool[http-worker]@5fbaf2de{STOPPING,2<=20<=200,i=0,q=12}[org.eclipse.jetty.util.thread.TryExecutor$$Lambda$280/1814466511@18707d06] Couldn't stop Thread[http-worker-59,5,main] |
|
2023-11-21T00:58:06.843-0600 WARN TestNG-test=Surefire test-1 com.facebook.presto.testng.services.LogTestDurationListener Tests from com.facebook.presto.verifier.framework.TestDataVerification took 1.88m |
|
@tdcmeehan please review |
presto-main/src/main/java/com/facebook/presto/type/TimestampWithTimeZoneOperators.java
Outdated
Show resolved
Hide resolved
|
@tdcmeehan please helpt o review |
Description
Eliminate unneeded else condition check
Motivation and Context
Streamlining and eliminating superfluous if-else conditions.
Impact
No impact, just removal of extraneous elements.
Test Plan
Execute unit tests in presto-main and verify castToTimestamp and castToTime scenarios.
Contributor checklist
Release Notes
Please follow release notes guidelines and fill in the release notes below.