Allowed space between number and unit in time span#387

Open

bhavnajindal wants to merge 591 commits intoKusto-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
bug_fix_1490

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

mcmajam
authored and
ltrk2
committed
Adding math functions

mcmajam
authored and
ltrk2
committed
use .size()

mcmajam
authored and
ltrk2
committed
fix summarize_exception_bug_2113

SadiHassan
authored and
ltrk2
committed
bug_fix_1413

SadiHassan
authored and
ltrk2
committed
fixed style error

SadiHassan
authored and
ltrk2
committed
rebased and func test added

SadiHassan
authored and
ltrk2
committed
Support multi-token needles
ltrk2
committed
Update unit tests
ltrk2
committed
Fix style issues
ltrk2
committed
Fix a linker error
ltrk2
committed
bug_fix_2159

SadiHassan
authored and
ltrk2
committed
rebased and code updated

SadiHassan
authored and
ltrk2
committed
rebase and address review comments

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

SadiHassan
authored and
ltrk2
committed
Addressed review comments

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

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

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

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

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

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

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

kashwy
authored and
ltrk2
committed
Fix linker error for good
ltrk2
committed
lookup() function

SadiHassan
authored and
ltrk2
committed
removed unused code

SadiHassan
authored and
ltrk2
committed
Addressed review comments

SadiHassan
authored and
ltrk2
committed
fix style issue

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

bemitc
authored and
ltrk2
committed
address trailing whitespace and ErrorCodes errors

bemitc
authored and
ltrk2
committed
add one additional missing ErrorCode

bemitc
authored and
ltrk2
committed
space after if and for

bemitc
authored and
ltrk2
committed
eliminate camelcase in variable names

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

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

bemitc
authored and
ltrk2
committed
make args_list const

bemitc
authored and
ltrk2
committed
handle columns with multiple rows correctly

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

bemitc
authored and
ltrk2
committed
fix style

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

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

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

bemitc
authored and
ltrk2
committed
size_t to const auto

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

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

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

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

bemitc
and
ltrk2
committed
resolution of several minor code review comments

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

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

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

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

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

bemitc
authored and
ltrk2
committed
update ip prefix heuristic

bemitc
authored and
ltrk2
committed
add #pragma once

bemitc
authored and
ltrk2
committed
fix typo

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

bemitc
authored and
ltrk2
committed
add release notes

bemitc
authored and
ltrk2
committed
Implement KQL getschema
ltrk2
committed
Finalize KQL getschema
ltrk2
committed
Resolve findings from style check
ltrk2
committed
string functions wip

SadiHassan
authored and
ltrk2
committed
string functions updated

SadiHassan
authored and
ltrk2
committed
fix style check

SadiHassan
authored and
ltrk2
committed
Addressed review comments

SadiHassan
authored and
ltrk2
committed
bug fix 1333 resubmit

SadiHassan
authored and
ltrk2
committed
rebased and addressed review comments

SadiHassan
authored and
ltrk2
committed
Bug fix 2167

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

SadiHassan
authored and
ltrk2
committed
added alias in unit test

SadiHassan
authored and
ltrk2
committed
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
Kusto-phase3: fixed kql range issues

kashwy
authored and
ltrk2
committed
bug fix 2424

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

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

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

bemitc
authored and
ltrk2
committed
add const to some params

bemitc
authored and
ltrk2
committed
add functional and unit tests

bemitc
authored and
ltrk2
committed
update unit test

bemitc
authored and
ltrk2
committed
format

bemitc
authored and
ltrk2
committed
fix mistake in unit test

bemitc
authored and
ltrk2
committed
remove ErrorCodes::NOT_IMPLEMENTED

bemitc
authored and
ltrk2
committed
eliminate variables in unit tests

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

bemitc
authored and
ltrk2
committed
move some duplicated code to header

bemitc
authored and
ltrk2
committed
reduce duplicate code

bemitc
authored and
ltrk2
committed
add static_cast

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

bemitc
authored and
ltrk2
committed
use std::numeric_limits

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

bemitc
authored and
ltrk2
committed
initialize with an empty string

bemitc
authored and
ltrk2
committed
do initialization inside if

bemitc
authored and
ltrk2
committed
check for full match

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

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

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

bemitc
authored and
ltrk2
committed
switch from list to sequence

bemitc
authored and
ltrk2
committed
restructure code to be more maintainable

bemitc
authored and
ltrk2
committed
cleanup minor bool issues

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

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

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

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

bemitc
and
ltrk2
committed
some minor changes suggested by review

bemitc
authored and
ltrk2
committed
remove std::move

bemitc
authored and
ltrk2
committed
Update functional tests
ltrk2
committed
Update release notes
ltrk2
committed
function 2395

SadiHassan
authored and
ltrk2
committed
native version implemented

SadiHassan
authored and
ltrk2
committed
updated tests

SadiHassan
authored and
ltrk2
committed
some minor changes

SadiHassan
authored and
ltrk2
committed
style correction

SadiHassan
authored and
ltrk2
committed
Revert addition of poco contrib folder
ltrk2
committed
Update release notes
ltrk2
committed
Core review changes

mcmajam
authored and
ltrk2
committed
Fix test

mcmajam
authored and
ltrk2
committed
additional code review changes

mcmajam
authored and
ltrk2
committed
Code review changes #3

mcmajam
authored and
ltrk2
committed
Update readme after release.

mcmajam
authored and
ltrk2
committed
format corrections

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

SadiHassan
authored and
ltrk2
committed
code updated

SadiHassan
authored and
ltrk2
committed
fixed unit test

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

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

kashwy
authored and
ltrk2
committed
Implement KQL toscalar
ltrk2
committed
Add further functional tests
ltrk2
committed
Update release notes
ltrk2
committed
Remove unused include
ltrk2
committed
Add a unit test
ltrk2
committed
Revert unnecessary changes
ltrk2
committed
Kusto-phase3: fixed the alias position to enable alias inside function

kashwy
authored and
ltrk2
committed
Correct type mapping for arrays
ltrk2
committed
Implement KQL query-wide default column naming scheme
ltrk2
committed
Get rid of unnecessary changes
ltrk2
committed
Remove unused members
ltrk2
committed
not function

SadiHassan
authored and
ltrk2
committed
updated release note

SadiHassan
authored and
ltrk2
committed
between function work in progress

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

SadiHassan
authored and
ltrk2
committed
updated test cases

SadiHassan
authored and
ltrk2
committed
functional tests

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

SadiHassan
authored and
ltrk2
committed
update release notes

SadiHassan
authored and
ltrk2
committed
removed change from Lexer.cpp

SadiHassan
authored and
ltrk2
committed
updated native function + parser

SadiHassan
authored and
ltrk2
committed
unit test separated

SadiHassan
authored and
ltrk2
committed
fix unit test for between function

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

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

kashwy
authored and
ltrk2
committed
KQL URL parse using x3

mcmajam
authored and
ltrk2
committed
Code review 1

mcmajam
authored and
ltrk2
committed
Additional changes

mcmajam
authored and
ltrk2
committed
More changes

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

mcmajam
authored and
ltrk2
committed
And again

mcmajam
authored and
ltrk2
committed
format

mcmajam
authored and
ltrk2
committed
make_string function

SadiHassan
authored and
ltrk2
committed
removed commented out codes

SadiHassan
authored and
ltrk2
committed
native function implemented

SadiHassan
authored and
ltrk2
committed
added unit test

SadiHassan
authored and
ltrk2
committed
added test case with function

SadiHassan
authored and
ltrk2
committed
code improvement and test case added

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

SadiHassan
authored and
ltrk2
committed
addressed review comments

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

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

bemitc
authored and
ltrk2
committed
release notes

bemitc
authored and
ltrk2
committed
update release notes

bemitc
authored and
ltrk2
committed
bug_fix_2520 (new_guid)

SadiHassan
authored and
ltrk2
committed
addressed review comments

SadiHassan
authored and
ltrk2
committed
Fix flaky functional test
ltrk2
committed
Fix epoch related issue

mcmajam
authored and
ltrk2
committed
Code reivew changes

mcmajam
authored and
ltrk2
committed
isascii() function

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

SadiHassan
authored and
ltrk2
committed
rebase and addressed review comments

SadiHassan
authored and
ltrk2
committed
bug_fix_2512

SadiHassan
authored and
ltrk2
committed
added func test

SadiHassan
authored and
ltrk2
committed
updated func test

SadiHassan
authored and
ltrk2
committed
bug_fix_2511

SadiHassan
authored and
ltrk2
committed
addressed review comments

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

bemitc
authored and
ltrk2
committed
add unit and functional tests

bemitc
authored and
ltrk2
committed
add release notes

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

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

bemitc
authored and
ltrk2
committed
move bracket_count outside loop

bemitc
authored and
ltrk2
committed
isutf8 function implemented

SadiHassan
authored and
ltrk2
committed
rebased and updated release note

SadiHassan
authored and
ltrk2
committed
resolve conflict

SadiHassan
authored and
ltrk2
committed
not native function

SadiHassan
authored and
ltrk2
committed
not native updated

SadiHassan
authored and
ltrk2
committed
addressed review comments

SadiHassan
authored and
ltrk2
committed
bug fix 2616

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

kashwy
authored and
ltrk2
committed
Deliver bug fixes

mcmajam
authored and
ltrk2
committed
Code review changes

mcmajam
authored and
ltrk2
committed
Add test file change

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

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

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

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

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

kashwy
authored and
ltrk2
committed
correct countof to handle overlapping strings

bemitc
authored and
ltrk2
committed
remove some headers and reformat

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

bemitc
and
ltrk2
committed
Kusto-phase3: updated release note

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

mcmajam
authored and
ltrk2
committed
ADd reame update

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

bhavnajindal
authored and
ltrk2
committed
Added gtest

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

bhavnajindal
authored and
ltrk2
committed
Add bhavnjindal as trusted contributor

bhavnajindal
authored and
ltrk2
committed
changed dcount implementation to use uniqueCombine64

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

bhavnajindal
authored and
ltrk2
committed
Automatic style fix

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

bhavnajindal
authored and
ltrk2
committed
Added functional and unit test

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

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

bhavnajindal
authored and
ltrk2
committed
addressed review comments by refactoring code

bhavnajindal
authored and
ltrk2
committed
code formatting

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

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

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

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

bhavnajindal
authored and
ltrk2
committed
Fix compile errors
ltrk2
committed
Get rid of kusto_auto
ltrk2
committed
Fix as many functional tests as possible
ltrk2
committed
Update unit tests
ltrk2
committed

Commits on Aug 4, 2023

Make hasTokenOrNull return null on empty needle
ltrk2
committed

Commits on Aug 16, 2023

Commits on Aug 17, 2023

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

Commits on Aug 22, 2023

Commits on Aug 28, 2023

Allowed space between number and unit in time span
bhavnajindal
committed