Update Scalar_functions.md

[Adez017]

Which issue does this PR close?

• Closes [Docs Fix]: Missing examples for Scalar Fucntions in docs #17004

Rationale for this change

while Browsing through the documentation i find the example for the scalar functions are missing

What changes are included in this PR?

Added examples for documentation

[Adez017]

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

my system doesnt updates the files . please can you do that

[alamb]

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 example, to find abs search for "Returns the absolute value of a number." like this:

https://github.com/search?q=repo%3Aapache%2Fdatafusion+%22Returns+the+absolute+value+of+a+number.%22&type=code

If you are having problems running update_docs perhaps you can try using the codespace:

[Adez017]

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 example, to find abs search for "Returns the absolute value of a number." like this:

https://github.com/search?q=repo%3Aapache%2Fdatafusion+%22Returns+the+absolute+value+of+a+number.%22&type=code

If you are having problems running update_docs perhaps you can try using the codespace:

just a quick question , does i need to install all dependencies over there or this are preconfigured

[alamb]

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 in the container

apt-get install npm

(I made a PR to add this explicitly to the container here #17019)

[comphead]

@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 documentation methods and creates an md file on top of that.

Do you see any issues with the tool?

[Adez017]

@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 documentation methods and creates an md file on top of that.

Do you see any issues with the tool?

i haven't used yet , will do it later at EOD

[Adez017]

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 in the container

apt-get install npm

(I made a PR to add this explicitly to the container here #17019)

Not working ,

[Adez017]

@comphead , @alamb , kindly take a look

[Adez017]

Hi @alamb , i fired the ./dev/update_doc.sh but it only updates the aggregate_function.md file not the scalar part.
also it shows like this

[comphead]

npm i -g npx
npm i -g prettier

[Adez017]

npm i -g prettier

[Adez017]

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

[comphead]

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 be easier to run on local machine and follow https://docs.npmjs.com/downloading-and-installing-node-js-and-npm

[alamb]

Not working ,

Perhaps you need to run apt-get update first to update the package registry

[Adez017]

Hi @alamb ,i think this is ready now , a bit late but everything is working now .
CC: @comphead

[Adez017]

any Updates @alamb

[Adez017]

Any Updates @alamb

[comphead]

@Adez017 I'll take a look today

[comphead]

not sure why this is here, was this file created manually?

[Adez017]

i think so , because later that time the build commands are halting

[Adez017]

Any Updates?? @comphead

[comphead]

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

[Adez017]

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 need to be fixed could you please explain