You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/09 18:51:51 UTC

[GitHub] [arrow-rs] tustvold opened a new issue, #3069: Debug format for timestamp ignores timezone

tustvold opened a new issue, #3069:
URL: https://github.com/apache/arrow-rs/issues/3069

   **Describe the bug**
   <!--
   A clear and concise description of what the bug is.
   -->
   
   The `Debug` implementation for `PrimitiveArray` ignores the timestamp of timestamp arrays.
   
   
   **To Reproduce**
   <!--
   Steps to reproduce the behavior:
   -->
   
   ```
   let a = TimestampSecondArray::from(vec![Some(86405), Some(1), None])
        .with_timezone("+01:00".to_string());
   
       println!("{:?}", a);
   ```
   
   Returns
   
   ```
   PrimitiveArray<Timestamp(Second, Some("+01:00"))>
   [
     1970-01-02T00:00:05,
     1970-01-01T00:00:01,
     null,
   ]
   ```
   
   Whereas
   
   ```
   let a = Arc::new(TimestampSecondArray::from(vec![Some(86405), Some(1), None])
               .with_timezone("+01:00".to_string())) as ArrayRef;
       let t = cast(&a, &DataType::Utf8).unwrap();
       println!("{:?}", t);
   ```
   
   Returns
   
   ```
   StringArray
   [
     "1970-01-02 01:00:05 +01:00",
     "1970-01-01 01:00:01 +01:00",
     null,
   ]
   ```
   
   **Expected behavior**
   <!--
   A clear and concise description of what you expected to happen.
   -->
   
   The string cast was fixed by @viirya in #2608
   
   **Additional context**
   <!--
   Add any other context about the problem here.
   -->
   
   FYI @waitingkuo 


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

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

[GitHub] [arrow-rs] tustvold commented on issue #3069: Debug format for timestamp ignores timezone

Posted by GitBox <gi...@apache.org>.
tustvold commented on issue #3069:
URL: https://github.com/apache/arrow-rs/issues/3069#issuecomment-1309273699

   Aah yes closing as duplicate of #2917
   
   And I at least found the inconsistency very confusing


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-rs] alamb commented on issue #3069: Debug format for timestamp ignores timezone

Posted by GitBox <gi...@apache.org>.
alamb commented on issue #3069:
URL: https://github.com/apache/arrow-rs/issues/3069#issuecomment-1312141634

   `label_issue.py` automatically added labels {'arrow'} from #3016


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-rs] waitingkuo commented on issue #3069: Debug format for timestamp ignores timezone

Posted by GitBox <gi...@apache.org>.
waitingkuo commented on issue #3069:
URL: https://github.com/apache/arrow-rs/issues/3069#issuecomment-1309271992

   i can submit a pr for this if showing timestamp with time zone shift is preffered 


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-rs] waitingkuo commented on issue #3069: Debug format for timestamp ignores timezone

Posted by GitBox <gi...@apache.org>.
waitingkuo commented on issue #3069:
URL: https://github.com/apache/arrow-rs/issues/3069#issuecomment-1309271050

   @tustvold 
   i think this is similar as #2917 
   i only fixed the prettyprint one but not the debut information because i'm not sure is it intended to let the user to see the underline timestamp 


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-rs] tustvold closed issue #3069: Debug format for timestamp ignores timezone

Posted by GitBox <gi...@apache.org>.
tustvold closed issue #3069: Debug format for timestamp ignores timezone
URL: https://github.com/apache/arrow-rs/issues/3069


-- 
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: github-unsubscribe@arrow.apache.org

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