Skip to content

YQL-17439: block implementation of From/IntervalFrom/To Datetime UDF functions #1755

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Feb 13, 2024

Conversation

fedor-miron
Copy link
Contributor

Added optimized block implementation of FromSeconds, FromMilliseconds, FromMicroseconds, IntervalFromDays, IntervalFromHours, IntervalFromMinutes, IntervalFromSeconds, IntervalFromMilliseconds, IntervalFromMicroseconds, ToDays, ToHours, ToMinutes.

@fedor-miron fedor-miron requested a review from a team as a code owner February 9, 2024 01:40
Copy link

github-actions bot commented Feb 9, 2024

2024-02-09 01:43:43 UTC Pre-commit check for 1eea081 has started.
2024-02-09 01:43:46 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-09 01:46:39 UTC Build successful.
2024-02-09 01:46:52 UTC Tests are running...
🔴 2024-02-09 01:56:04 UTC Test run completed, no test results found for commit d6d09e4. Please check build logs.
2024-02-09 01:56:09 UTC Check cancelled

Copy link

github-actions bot commented Feb 9, 2024

2024-02-09 01:44:04 UTC Pre-commit check for 1eea081 has started.
2024-02-09 01:44:07 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-09 01:46:39 UTC Build successful.
2024-02-09 01:46:51 UTC Tests are running...
🔴 2024-02-09 01:55:55 UTC Test run completed, no test results found for commit d6d09e4. Please check build logs.
2024-02-09 01:55:57 UTC Check cancelled

Copy link

github-actions bot commented Feb 9, 2024

2024-02-09 01:59:36 UTC Pre-commit check for 4b9b1ad has started.
2024-02-09 01:59:39 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-09 02:01:25 UTC Build successful.
2024-02-09 02:01:40 UTC Tests are running...
🔴 2024-02-09 03:49:49 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67408 56470 0 14 10892 32

Copy link

github-actions bot commented Feb 9, 2024

2024-02-09 01:59:41 UTC Pre-commit check for 4b9b1ad has started.
2024-02-09 01:59:43 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-09 02:01:26 UTC Build successful.
2024-02-09 02:01:41 UTC Tests are running...
🔴 2024-02-09 03:41:27 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14738 14544 0 21 134 39

bool isAllNull = inArray->GetNullCount() == length;
if (!isAllNull) {
for (i64 i = 0; i < length; ++i) {
outValues[i] = Core(inValues[i]);
Copy link
Collaborator

@vitstn vitstn Feb 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please return std::pair from Core, see https://godbolt.org/z/coYsffa5T

@@ -862,8 +869,47 @@ NUdf::TUnboxedValuePod DoAddYears(const NUdf::TUnboxedValuePod& date, i64 years,
};

// From*

template<typename TInput, typename TOutput, TOutput Core(TInput), bool Filter(TInput)>
struct TUnaryFixedSizeKernelWithFilter {
Copy link
Collaborator

@vitstn vitstn Feb 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add suffix Unsafe and ensure that TInput/TOutput are std::is_arithmetic

Copy link

github-actions bot commented Feb 11, 2024

2024-02-11 21:40:46 UTC Pre-commit check for ca088f2 has started.
2024-02-11 21:40:47 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-11 21:43:38 UTC Build successful.
2024-02-11 21:43:47 UTC Tests are running...
🔴 2024-02-11 23:35:08 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67419 56474 0 17 10892 36

Copy link

github-actions bot commented Feb 11, 2024

2024-02-11 21:40:46 UTC Pre-commit check for ca088f2 has started.
2024-02-11 21:40:48 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-11 21:43:15 UTC Build successful.
2024-02-11 21:43:29 UTC Tests are running...
🔴 2024-02-11 23:29:08 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14743 14537 0 23 149 34

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 14:04:48 UTC Pre-commit check for c7cf747 has started.
2024-02-12 14:04:49 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-12 14:08:13 UTC Build successful.
2024-02-12 14:08:26 UTC Tests are running...
🔴 2024-02-12 15:59:04 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67429 56366 0 17 10970 76

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 14:04:55 UTC Pre-commit check for c7cf747 has started.
2024-02-12 14:04:56 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-12 14:07:17 UTC Build successful.
2024-02-12 14:07:29 UTC Tests are running...
🔴 2024-02-12 15:54:30 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14753 14511 0 18 177 47

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 23:07:09 UTC Pre-commit check for 0bf510a has started.
2024-02-12 23:07:11 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-12 23:09:33 UTC Build successful.
2024-02-12 23:09:44 UTC Tests are running...
🔴 2024-02-12 23:15:40 UTC Test run completed, no test results found for commit 3d53296. Please check build logs.
2024-02-12 23:15:44 UTC Check cancelled

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 23:07:20 UTC Pre-commit check for 0bf510a has started.
2024-02-12 23:07:22 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-12 23:09:55 UTC Build successful.
2024-02-12 23:10:03 UTC Tests are running...
🔴 2024-02-12 23:15:39 UTC Test run completed, no test results found for commit 3d53296. Please check build logs.
2024-02-12 23:15:41 UTC Check cancelled

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 23:16:19 UTC Pre-commit check for 07977b2 has started.
2024-02-12 23:16:20 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-12 23:17:29 UTC Build successful.
2024-02-12 23:17:40 UTC Tests are running...
🔴 2024-02-13 01:02:27 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14756 14526 0 20 154 56

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 23:16:26 UTC Pre-commit check for 07977b2 has started.
2024-02-12 23:16:28 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-12 23:17:41 UTC Build successful.
2024-02-12 23:17:50 UTC Tests are running...
🔴 2024-02-13 01:05:59 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67444 56441 0 16 10956 31

Copy link

github-actions bot commented Feb 13, 2024

2024-02-13 09:18:51 UTC Pre-commit check for 182c28c has started.
2024-02-13 09:18:53 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-13 09:21:06 UTC Build successful.
2024-02-13 09:21:16 UTC Tests are running...
🔴 2024-02-13 10:59:18 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14759 14574 0 20 131 34

Copy link

github-actions bot commented Feb 13, 2024

2024-02-13 09:18:53 UTC Pre-commit check for 182c28c has started.
2024-02-13 09:18:55 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-13 09:21:47 UTC Build successful.
2024-02-13 09:21:57 UTC Tests are running...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants