You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2020/07/20 09:23:15 UTC
[GitHub] [incubator-doris] vagetablechicken opened a new issue #4121: [UT] TestTimestampedVersionTracker assumes the local tz is UTC+8
vagetablechicken opened a new issue #4121:
URL: https://github.com/apache/incubator-doris/issues/4121
**Describe the bug**
`TimestampedVersionTracker::get_stale_version_path_json_doc()` will return time string by using the local time.
We just assume that the local time zone is utc+8, so the expect_result "last create time" is in utc+8.
https://github.com/apache/incubator-doris/blob/fbf7bd6a1de9caefa58f122e91f1387430df870c/be/test/olap/timestamped_version_tracker_test.cpp#L773-L778
**To Reproduce**
Steps to reproduce the behavior:
1. tzselect & export another TZ
2. run ut `TestTimestampedVersionTracker`
**Expected behavior**
Maybe we should adjust the expect result for the local tz in machine, or improve`TimestampedVersionTracker::get_stale_version_path_json_doc()`.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [incubator-doris] vagetablechicken closed issue #4121: [UT] TestTimestampedVersionTracker assumes the local tz is UTC+8
Posted by GitBox <gi...@apache.org>.
vagetablechicken closed issue #4121:
URL: https://github.com/apache/incubator-doris/issues/4121
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [incubator-doris] vagetablechicken closed issue #4121: [UT] TestTimestampedVersionTracker assumes the local tz is UTC+8
Posted by GitBox <gi...@apache.org>.
vagetablechicken closed issue #4121:
URL: https://github.com/apache/incubator-doris/issues/4121
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [incubator-doris] vagetablechicken closed issue #4121: [UT] TestTimestampedVersionTracker assumes the local tz is UTC+8
Posted by GitBox <gi...@apache.org>.
vagetablechicken closed issue #4121:
URL: https://github.com/apache/incubator-doris/issues/4121
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org