Jefffrey commented on code in PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#discussion_r2347671241
##
datafusion/functions/src/math/monotonicity.rs:
##
@@ -309,6 +416,16 @@ static DOCUMENTATION_DEGREES: LazyLock =
LazyLock::new(|| {
"degrees(numeric_e
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3238942382
> To make this PR mergeable it is needed to fix .md file. Otherwise it would
fail the CI, you can try doing this in github codespace if local machine cannot
run scripts
what n
comphead commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3238572979
To make this PR mergeable it is needed to fix .md file. Otherwise it would
fail the CI, you can try doing this in github codespace if local machine cannot
run scripts
--
This is
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3237831494
Any Updates?? @comphead
--
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
Adez017 commented on code in PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#discussion_r2305845680
##
docs/source/user-guide/sql/scalar_functions.md:
##
@@ -527,6 +901,17 @@ tanh(numeric_expression)
- **numeric_expression**: Numeric expression to operate on.
comphead commented on code in PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#discussion_r2305718642
##
docs/source/user-guide/sql/scalar_functions.md:
##
@@ -527,6 +901,17 @@ tanh(numeric_expression)
- **numeric_expression**: Numeric expression to operate on.
comphead commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3228905382
@Adez017 I'll take a look today
--
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 t
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3228870673
Any Updates @alamb
--
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 comme
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3220888464
any Updates @alamb
--
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 comme
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3218160323
Hi @alamb ,i think this is ready now , a bit late but everything is working
now .
CC: @comphead
--
This is an automated message from the Apache Git Service.
To respond to the
alamb commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3165625411
> Not working ,
Perhaps you need to run `apt-get update` first to update the package registry
--
This is an automated message from the Apache Git Service.
To respond to the
comphead commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3160738761
> Hi @comphead , could you give it a try in the codespace , like if the
changes are correct or note also i would like to mentioned that check the
commits also
I feel it would
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3160707098
Hi @comphead , could you give it a try in the codespace , like if the
changes are correct or note also i would like to mentioned that check the
commits also
--
This is an automat
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3160698799
> npm i -g prettier
https://github.com/user-attachments/assets/691a7f7e-cce0-4df2-8c57-36220293ea3a";
/>
--
This is an automated message from the Apache Git Service.
To
comphead commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3160617986
`npm i -g prettier`
--
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 comm
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3159882354
Hi @alamb , i fired the `./dev/update_doc.sh` but it only updates the
aggregate_function.md file not the scalar part
--
This is an automated message from the Apache Git Service.
T
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3159810610
@comphead , @alamb , kindly take a look
--
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 g
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3159804268
> > just a quick question , does i need to install all dependencies over
there or this are preconfigured
>
> they are pre-configured mostly
>
> You may have to run this
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3157011483
> @Adez017 this document is autogenerated by the
`./dev/update_function_docs.sh` script which runs separate rust console app to
gather documentation from scalar functions documentati
comphead commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3156986509
@Adez017 this document is autogenerated by the
`./dev/update_function_docs.sh` script which runs separate rust console app to
gather documentation from scalar functions documentatio
alamb commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3155888627
> just a quick question , does i need to install all dependencies over there
or this are preconfigured
they are pre-configured mostly
You may have to run this command
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3155055953
> Hi @Adez017
>
> > hi @alamb , i coudnt find any rust file for this function .
>
> I think you can find the corresponding rust file by searching for the
function descri
alamb commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3146420097
Hi @Adez017
> hi @alamb , i coudnt find any rust file for this function .
I think you can find the corresponding rust file by searching for the
function description. For
Adez017 commented on PR #17018:
URL: https://github.com/apache/datafusion/pull/17018#issuecomment-3146276212
hi @alamb , i coudnt find any rust file for this function . directly added
the examples to the markdown . need a little help , while firing
``` ./dev/update_function_docs.sh
`
Adez017 opened a new pull request, #17018:
URL: https://github.com/apache/datafusion/pull/17018
## Which issue does this PR close?
- Closes #17004
## Rationale for this change
while Browsing through the documentation i find the example for the scalar
functi
alamb commented on PR #12627:
URL: https://github.com/apache/datafusion/pull/12627#issuecomment-2374828911
Thanks again @Abdullahsab3
--
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
alamb merged PR #12627:
URL: https://github.com/apache/datafusion/pull/12627
--
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-unsubscr...@datafusi
Abdullahsab3 opened a new pull request, #12627:
URL: https://github.com/apache/datafusion/pull/12627
Remove extra space in reference to list_any_value
## Which issue does this PR close?
Closes #.
## Rationale for this change
## What changes are incl
28 matches
Mail list logo
