AlenkaF commented on code in PR #49694:
URL: https://github.com/apache/arrow/pull/49694#discussion_r3063532764


##########
python/pyarrow/scalar.pxi:
##########
@@ -822,7 +822,14 @@ cdef class TimestampScalar(Scalar):
             return None
 
         if not dtype.timezone().empty():
-            tzinfo = string_to_tzinfo(frombytes(dtype.timezone()))
+            prefer_zoneinfo = True
+            # only we this method would return a pandas.Timestamp, prefer

Review Comment:
   ```suggestion
               # only if this method would return a pandas.Timestamp, prefer
   ```
   
   Note, I needed some time understanding what is meant here. Maybe simpler is 
better?
   
   ```
               # Adjust preference based on the pandas version
   ```
   
   or a bit more:
   
   ```
               # Adjust preference based on the pandas version -
               # keep returning pytz for older pandas
   ```



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

Reply via email to