Allowed space between number and unit in time span - #387

Open

bhavnajindal wants to merge 591 commits into

Kusto-phase3from

I3149

This pull request is big! We're only showing the most recent 250 commits

Commits on Aug 3, 2023

Kusto-phase3: fix mv-expand token index issue

kashwy
authored and
ltrk2
committed
af2ed2b
bug_fix_1490

SadiHassan
authored and
ltrk2
committed
4ad852b
Handle multi-word needles with has operators
ltrk2
committed
a592733
Update release notes
ltrk2
committed
a41966c
Fixing empty functions

mcmajam
authored and
ltrk2
committed
449162e
Adding math functions

mcmajam
authored and
ltrk2
committed
35fc0e9
use .size()

mcmajam
authored and
ltrk2
committed
d227b2b
fix summarize_exception_bug_2113

SadiHassan
authored and
ltrk2
committed
e93edf9
bug_fix_1413

SadiHassan
authored and
ltrk2
committed
bb4edff
fixed style error

SadiHassan
authored and
ltrk2
committed
9f09169
rebased and func test added

SadiHassan
authored and
ltrk2
committed
fa6b7f0
Support multi-token needles
ltrk2
committed
0884586
Update unit tests
ltrk2
committed
a009c39
Fix style issues
ltrk2
committed
785f70f
Fix a linker error
ltrk2
committed
549aeda
bug_fix_2159

SadiHassan
authored and
ltrk2
committed
48303ce
rebased and code updated

SadiHassan
authored and
ltrk2
committed
f7e5666
rebase and address review comments

SadiHassan
authored and
ltrk2
committed
5a6e54e
Attempt to supress a pylint finding
ltrk2
committed
6950443
variance related functions --- 2252

SadiHassan
authored and
ltrk2
committed
9af4fe8
Addressed review comments

SadiHassan
authored and
ltrk2
committed
039461f
Kusto-phase3 : fixed some sub-query issues.

kashwy
authored and
ltrk2
committed
b9294d3
Kusto-phase3: fixed style issue, count bugs

kashwy
authored and
ltrk2
committed
83771e3
Kusto-phase3 : fix typo in sub-query

kashwy
authored and
ltrk2
committed
b4c8826
Kusto-phase3: add KQL project-away operator

kashwy
authored and
ltrk2
committed
ffa6cdc
Kusto-phase3 : Add test for projectaway

kashwy
authored and
ltrk2
committed
13a3769
Kusto-phase3: update release note

kashwy
authored and
ltrk2
committed
73a0dcb
Kusto-phase3 : updated function to follow the style

kashwy
authored and
ltrk2
committed
445e99f
Fix linker error for good
ltrk2
committed
2938c97
lookup() function

SadiHassan
authored and
ltrk2
committed
b421ed8
removed unused code

SadiHassan
authored and
ltrk2
committed
6dedfc4
Addressed review comments

SadiHassan
authored and
ltrk2
committed
03c77c6
fix style issue

SadiHassan
authored and
ltrk2
committed
66392ec
Bring everything in line with v23.2
ltrk2
committed
a439cdd
implementation of has_ipv4, has_any_ipv4, has_ipv4_prefix, and has_any_ipv4_prefix

bemitc
authored and
ltrk2
committed
290c735
address trailing whitespace and ErrorCodes errors

bemitc
authored and
ltrk2
committed
a49b632
add one additional missing ErrorCode

bemitc
authored and
ltrk2
committed
2162a94
space after if and for

bemitc
authored and
ltrk2
committed
f319ad5
eliminate camelcase in variable names

bemitc
authored and
ltrk2
committed
5f34249
use direct mapping for non any functions and mop up some stray camelcase variables

bemitc
authored and
ltrk2
committed
cd159be
directMap remaining functions instead of a wrapper.

bemitc
authored and
ltrk2
committed
22b82d4
make args_list const

bemitc
authored and
ltrk2
committed
e75a17a
handle columns with multiple rows correctly

bemitc
authored and
ltrk2
committed
0b78f3e
use input_rows_count instead of calculating maximum size across arguments

bemitc
authored and
ltrk2
committed
577293b
fix style

bemitc
authored and
ltrk2
committed
3efc2c5
put lambda expression on a single line

bemitc
authored and
ltrk2
committed
377a5bf
use std::ranges::all_of instead of a for loop

bemitc
authored and
ltrk2
committed
c4192c6
clean up functional tests and add unit tests

bemitc
authored and
ltrk2
committed
3825964
size_t to const auto

bemitc
authored and
ltrk2
committed
fa72315
change some uses of std::string to auto

bemitc
authored and
ltrk2
committed
396ea7a
row count is the same across all arguments, so simplify the argument extraction a bit

bemitc
authored and
ltrk2
committed
4b19676
const a lambda variable and update functional test and baseline

bemitc
authored and
ltrk2
committed
4e723d4
Update src/Functions/Kusto/kqlHasAnyIpv4.cpp

bemitc
and
ltrk2
committed
0e9506a
resolution of several minor code review comments

bemitc
authored and
ltrk2
committed
2042d88
share argument verification across kqlHasAnyIpv4.cpp and kqlHasAnyIpv4Prefix.cpp

bemitc
authored and
ltrk2
committed
4f6a6ed
convert one remaining post increment to pre increment

bemitc
authored and
ltrk2
committed
48d1502
split executeImpl up to make it easier to understand

bemitc
authored and
ltrk2
committed
38b88d2
improve heuristic for ip prefixes and avoid a temp std::string creation on every evaluated ip

bemitc
authored and
ltrk2
committed
8695f23
add updated heuristic to the array case as well

bemitc
authored and
ltrk2
committed
2e2ffce
update ip prefix heuristic

bemitc
authored and
ltrk2
committed
2e4dd5f
add #pragma once

bemitc
authored and
ltrk2
committed
25fdb64
fix typo

bemitc
authored and
ltrk2
committed
bb1e8f6
format kqlHasAnyIp.h and kqlHasAnyIpv4.cpp

bemitc
authored and
ltrk2
committed
eceb9da
add release notes

bemitc
authored and
ltrk2
committed
836499a
Implement KQL getschema
ltrk2
committed
f1d5edc
Finalize KQL getschema
ltrk2
committed
67177b9
Resolve findings from style check
ltrk2
committed
860163b
string functions wip

SadiHassan
authored and
ltrk2
committed
be17029
string functions updated

SadiHassan
authored and
ltrk2
committed
275488b
fix style check

SadiHassan
authored and
ltrk2
committed
5620705
Addressed review comments

SadiHassan
authored and
ltrk2
committed
f04f3a7
bug fix 1333 resubmit

SadiHassan
authored and
ltrk2
committed
e9f1780
rebased and addressed review comments

SadiHassan
authored and
ltrk2
committed
be22c85
Bug fix 2167

SadiHassan
authored and
ltrk2
committed
c528dab
Implement default aliasing for KQL print
ltrk2
committed
d064f94
bug_fix_2161

SadiHassan
authored and
ltrk2
committed
6036b6c
added alias in unit test

SadiHassan
authored and
ltrk2
committed
800160c
Kusto-phase3: fixed the bug core dump if query end with pipe, force print nad range to be the first operator

kashwy
authored and
ltrk2
committed
1c7f6b5
Kusto-phase3: fixed kql range issues

kashwy
authored and
ltrk2
committed
dfc51e8
bug fix 2424

SadiHassan
authored and
ltrk2
committed
0aac4b6
Kusto-phase3 : fix sort issues

kashwy
authored and
ltrk2
committed
4a6afac
Kusto-phase3: fix typo for sort issue

kashwy
authored and
ltrk2
committed
2a8c27b
implementation of has_ipv6(), has_any_ipv6(), has_ipv6_prefix(), and has_any_ipv6_prefix()

bemitc
authored and
ltrk2
committed
36629d6
add const to some params

bemitc
authored and
ltrk2
committed
22c7643
add functional and unit tests

bemitc
authored and
ltrk2
committed
6c2331d
update unit test

bemitc
authored and
ltrk2
committed
d9fe9b7
format

bemitc
authored and
ltrk2
committed
7ed5fc4
fix mistake in unit test

bemitc
authored and
ltrk2
committed
bcd2974
remove ErrorCodes::NOT_IMPLEMENTED

bemitc
authored and
ltrk2
committed
bb14c9b
eliminate variables in unit tests

bemitc
authored and
ltrk2
committed
f48430b
undo vscode formatting in unit tests to improve readability

bemitc
authored and
ltrk2
committed
ef9869e
move some duplicated code to header

bemitc
authored and
ltrk2
committed
aeb01a0
reduce duplicate code

bemitc
authored and
ltrk2
committed
e032c95
add static_cast

bemitc
authored and
ltrk2
committed
cdfe2b1
use std::accumulate instead of std::all_of

bemitc
authored and
ltrk2
committed
55723bc
use std::numeric_limits

bemitc
authored and
ltrk2
committed
8f10ac1
use boost::spirit::x3::repeat

bemitc
authored and
ltrk2
committed
f5ccf72
initialize with an empty string

bemitc
authored and
ltrk2
committed
dacc0bf
do initialization inside if

bemitc
authored and
ltrk2
committed
42868bd
check for full match

bemitc
authored and
ltrk2
committed
3a3baad
clean up parsing of prefixes with embedded ipv4 addresses

bemitc
authored and
ltrk2
committed
c307b3d
Update src/Functions/Kusto/kqlHasAnyIp.cpp

bemitc
and
ltrk2
committed
016b30f
return std::optional<std::string>

bemitc
authored and
ltrk2
committed
9dc8436
switch from list to sequence

bemitc
authored and
ltrk2
committed
901ce5d
restructure code to be more maintainable

bemitc
authored and
ltrk2
committed
b455d01
cleanup minor bool issues

bemitc
authored and
ltrk2
committed
a651556
Update src/Functions/Kusto/kqlHasAnyIp.cpp

bemitc
and
ltrk2
committed
96d935d
Update src/Functions/Kusto/kqlHasAnyIp.cpp

bemitc
and
ltrk2
committed
6de6a2e
Update src/Functions/Kusto/kqlHasAnyIp.cpp

bemitc
and
ltrk2
committed
267905f
Update src/Functions/Kusto/kqlHasAnyIp.cpp

bemitc
and
ltrk2
committed
aa4fb3f
some minor changes suggested by review

bemitc
authored and
ltrk2
committed
43b6383
remove std::move

bemitc
authored and
ltrk2
committed
8b8894e
Update functional tests
ltrk2
committed
9783db5
Update release notes
ltrk2
committed
b85e140
function 2395

SadiHassan
authored and
ltrk2
committed
1b83cc1
native version implemented

SadiHassan
authored and
ltrk2
committed
40c62cd
updated tests

SadiHassan
authored and
ltrk2
committed
f45b2c8
some minor changes

SadiHassan
authored and
ltrk2
committed
b6821dd
style correction

SadiHassan
authored and
ltrk2
committed
ef798bb
Revert addition of poco contrib folder
ltrk2
committed
624f6bc
Update release notes
ltrk2
committed
422c562
Core review changes

mcmajam
authored and
ltrk2
committed
129024a
Fix test

mcmajam
authored and
ltrk2
committed
995f25e
additional code review changes

mcmajam
authored and
ltrk2
committed
7999e73
Code review changes #3

mcmajam
authored and
ltrk2
committed
6ceb3e2
Update readme after release.

mcmajam
authored and
ltrk2
committed
91fca6e
format corrections

mcmajam
authored and
ltrk2
committed
0230f90
bug fix 2174, 2175,1075(improvement)

SadiHassan
authored and
ltrk2
committed
7740b36
code updated

SadiHassan
authored and
ltrk2
committed
e30c2a7
fixed unit test

SadiHassan
authored and
ltrk2
committed
1bd8ecf
rebased and added more test cases with double quotes

SadiHassan
authored and
ltrk2
committed
4b7f162
Kusto-phase3 : enable range to be inside other funtion for a nested call

kashwy
authored and
ltrk2
committed
c1836cc
Implement KQL toscalar
ltrk2
committed
a78b0a3
Add further functional tests
ltrk2
committed
97659fd
Update release notes
ltrk2
committed
56905a4
Remove unused include
ltrk2
committed
3e56b20
Add a unit test
ltrk2
committed
4d9c4e9
Revert unnecessary changes
ltrk2
committed
10e4ced
Kusto-phase3: fixed the alias position to enable alias inside function

kashwy
authored and
ltrk2
committed
7cf5c07
Correct type mapping for arrays
ltrk2
committed
4f0d28a
Implement KQL query-wide default column naming scheme
ltrk2
committed
4556b9f
Get rid of unnecessary changes
ltrk2
committed
b2d8739
Remove unused members
ltrk2
committed
39cc522
not function

SadiHassan
authored and
ltrk2
committed
b886b40
updated release note

SadiHassan
authored and
ltrk2
committed
a9da9fc
between function work in progress

SadiHassan
authored and
ltrk2
committed
09e1a3a
solved floating point parsing issue by changing Lexer.cpp, but this change might not see the light in OSS

SadiHassan
authored and
ltrk2
committed
3ae2563
updated test cases

SadiHassan
authored and
ltrk2
committed
9fc91ec
functional tests

SadiHassan
authored and
ltrk2
committed
6306b41
floating point handled in kql operator cpp

SadiHassan
authored and
ltrk2
committed
b9536a3
update release notes

SadiHassan
authored and
ltrk2
committed
ceffaa9
removed change from Lexer.cpp

SadiHassan
authored and
ltrk2
committed
629fd07
updated native function + parser

SadiHassan
authored and
ltrk2
committed
55b5bb3
unit test separated

SadiHassan
authored and
ltrk2
committed
f8345e2
fix unit test for between function

SadiHassan
authored and
ltrk2
committed
5cc82a9
Kusto-phase3: enable make-series to use expression

kashwy
authored and
ltrk2
committed
e3861fa
Kusto-phase3 : format for fix make-series issues

kashwy
authored and
ltrk2
committed
b386061
KQL URL parse using x3

mcmajam
authored and
ltrk2
committed
a80f507
Code review 1

mcmajam
authored and
ltrk2
committed
197a4fc
Additional changes

mcmajam
authored and
ltrk2
committed
c1a3137
More changes

mcmajam
authored and
ltrk2
committed
d69c48a
Update readme, slight change in username password.

mcmajam
authored and
ltrk2
committed
ccae6e0
And again

mcmajam
authored and
ltrk2
committed
50d9042
format

mcmajam
authored and
ltrk2
committed
6676058
make_string function

SadiHassan
authored and
ltrk2
committed
6634679
removed commented out codes

SadiHassan
authored and
ltrk2
committed
e3cfb55
native function implemented

SadiHassan
authored and
ltrk2
committed
b1e2c74
added unit test

SadiHassan
authored and
ltrk2
committed
59a5831
added test case with function

SadiHassan
authored and
ltrk2
committed
0ac3bc2
code improvement and test case added

SadiHassan
authored and
ltrk2
committed
6f8d11e
addressed review coments, added all coverages of unicode, exception etc.

SadiHassan
authored and
ltrk2
committed
49ae441
addressed review comments

SadiHassan
authored and
ltrk2
committed
fd978be
Implement indexof_regex
ltrk2
committed
5419243
Use re2 instead of std::regex
ltrk2
committed
1fa246f
support multiple output columns in arg_max and arg_min

bemitc
authored and
ltrk2
committed
a80fe03
add unit test for arg_max() and arg_min()

bemitc
authored and
ltrk2
committed
5bcc10b
release notes

bemitc
authored and
ltrk2
committed
eef1d0e
update release notes

bemitc
authored and
ltrk2
committed
9fefd74
bug_fix_2520 (new_guid)

SadiHassan
authored and
ltrk2
committed
287033f
addressed review comments

SadiHassan
authored and
ltrk2
committed
96fcf1d
Fix flaky functional test
ltrk2
committed
4da0b50
Fix epoch related issue

mcmajam
authored and
ltrk2
committed
3847254
Code reivew changes

mcmajam
authored and
ltrk2
committed
1d37373
isascii() function

SadiHassan
authored and
ltrk2
committed
3c34ca7
using splitByString() instead of Regex

SadiHassan
authored and
ltrk2
committed
817ff72
rebase and addressed review comments

SadiHassan
authored and
ltrk2
committed
23c1e6a
bug_fix_2512

SadiHassan
authored and
ltrk2
committed
fb50a45
added func test

SadiHassan
authored and
ltrk2
committed
c2cb719
updated func test

SadiHassan
authored and
ltrk2
committed
53d5094
bug_fix_2511

SadiHassan
authored and
ltrk2
committed
a41a0d0
addressed review comments

SadiHassan
authored and
ltrk2
committed
ae8e6da
Implement a very toned-down version of hll functions
ltrk2
committed
1a85471
add project-rename operator

bemitc
authored and
ltrk2
committed
10dfd59
add unit and functional tests

bemitc
authored and
ltrk2
committed
c1b492e
add release notes

bemitc
authored and
ltrk2
committed
4a8e957
syntax error if any of the expressions lack an assignment

bemitc
authored and
ltrk2
committed
3bbcf9c
improve parsing, exception if we rename the same column multiple times to match adx

bemitc
authored and
ltrk2
committed
695e9c6
move bracket_count outside loop

bemitc
authored and
ltrk2
committed
5a4e100
isutf8 function implemented

SadiHassan
authored and
ltrk2
committed
c3bb8fd
rebased and updated release note

SadiHassan
authored and
ltrk2
committed
dc6a7c2
resolve conflict

SadiHassan
authored and
ltrk2
committed
8c16f10
not native function

SadiHassan
authored and
ltrk2
committed
38cbd51
not native updated

SadiHassan
authored and
ltrk2
committed
4ebe7e7
addressed review comments

SadiHassan
authored and
ltrk2
committed
2b5b9f3
bug fix 2616

SadiHassan
authored and
ltrk2
committed
9d88bdd
Implement default column naming scheme for KQL project
ltrk2
committed
fd3511c
Add another functional test case
ltrk2
committed
0acc4cb
Kusto-phase3 : Fix core dump when table is missing in pipeline

kashwy
authored and
ltrk2
committed
fe15ed0
Deliver bug fixes

mcmajam
authored and
ltrk2
committed
c2ab449
Code review changes

mcmajam
authored and
ltrk2
committed
6a57b55
Add test file change

mcmajam
authored and
ltrk2
committed
d096e83
throw an exception if arg_max or arg_min have fewer than 2 arguments

bemitc
authored and
ltrk2
committed
d4fc732
use getArguments() instead of walking the tokens

bemitc
authored and
ltrk2
committed
587aa35
use auto instead of std::string and eliminate a const that's not needed

bemitc
authored and
ltrk2
committed
faf14c7
use std::views::filter(...) instead of an if check

bemitc
authored and
ltrk2
committed
f6eaea2
Kusto-phase3: fix core dump when kql function is not complete

kashwy
authored and
ltrk2
committed
a24900c
correct countof to handle overlapping strings

bemitc
authored and
ltrk2
committed
17a5086
remove some headers and reformat

bemitc
authored and
ltrk2
committed
827df79
Update src/Functions/Kusto/kqlCountOverlappingSubstrings.cpp

bemitc
and
ltrk2
committed
0a4aa78
Kusto-phase3: updated release note

kashwy
authored and
ltrk2
committed
254ea03
Merge branch 'Kusto-phase3' of https://github.com/ClibMouse/ClickHouse into Kusto-phase3

mcmajam
authored and
ltrk2
committed
9e53bad
ADd reame update

mcmajam
authored and
ltrk2
committed
099d49d
#2609: fixed method to return NULL for invalid encoded strings

bhavnajindal
authored and
ltrk2
committed
e6d9a66
Added gtest

bhavnajindal
authored and
ltrk2
committed
d581fd5
Added uniqueIdentifier in the query and fixed the gtest accordingly

bhavnajindal
authored and
ltrk2
committed
f18f464
Add bhavnjindal as trusted contributor

bhavnajindal
authored and
ltrk2
committed
77e567f
changed dcount implementation to use uniqueCombine64

bhavnajindal
authored and
ltrk2
committed
e4b5843
Fixed the regex by adding anchor in Project-Away query

bhavnajindal
authored and
ltrk2
committed
bb518a6
Automatic style fix

robot-clickhouse
authored and
ltrk2
committed
0fe68eb
Moved changes to utility method, added functional test

bhavnajindal
authored and
ltrk2
committed
5d8311b
Added functional and unit test

bhavnajindal
authored and
ltrk2
committed
5b19229
Fixed formatting and added early return in method

bhavnajindal
authored and
ltrk2
committed
d93a51a
Fixed the wildcardToRegex method to return nullopt in case the the string does not contain any wildcard

bhavnajindal
authored and
ltrk2
committed
78d302d
addressed review comments by refactoring code

bhavnajindal
authored and
ltrk2
committed
e8299f7
code formatting

bhavnajindal
authored and
ltrk2
committed
43b2e4c
[3023]- Enable kql table function support both heredoc and regular format

bhavnajindal
authored and
ltrk2
committed
af43efb
Fixed a few unit test for DateTimeFunctions

bhavnajindal
authored and
ltrk2
committed
546b410
Kusto-phase3: update release node July

kashwy
authored and
ltrk2
committed
9857cb1
check for accepted values for accuracy in dcount,dcountif (#375)

bhavnajindal
authored and
ltrk2
committed
cd69539
Fix compile errors
ltrk2
committed
c0c8dee
Get rid of kusto_auto
ltrk2
committed
41aed6f
Fix as many functional tests as possible
ltrk2
committed
9b8e7f9
Update unit tests
ltrk2
committed
eada4b1

Commits on Aug 4, 2023

Make hasTokenOrNull return null on empty needle
ltrk2
committed
fc314b3

Commits on Aug 16, 2023

Commits on Aug 17, 2023

fix-3091 added simple mapping for constant expressions in array index(#382)
bhavnajindal
authored
d2d528d

Commits on Aug 22, 2023

Commits on Aug 28, 2023

Allowed space between number and unit in time span
bhavnajindal
committed
40fac9b

Uh oh!

Commits

Commits on Aug 3, 2023