support for bulk operations

[yelisap]

Hi Teams,

We have a use case which will need create/update multi records at one time.
Currently we are using follow scripts to save them.

cdsService.dispatch({
                event: "CREATE",
                path: this.entity.name,
                data: [{}, {}, {}],
                query: INSERT.into(this.entity).entries([{}, {}, {}])
            })

But we found it will not generate audit log information.

Is create/update multi records at one time is supported in @cap-js/audit-logging ?

Follow are some version of components we are using.

1. "@sap/cds": "7.8.2"
2. "@cap-js/audit-logging": "0.8.3"

Thanks for the helping on this issue.

Best Regards,
Ye Li

[sjvans]

hi @yelisap

no, this is currently not supported.

you'll need to write the logs yourself or do single inserts/ updates. please note that modification logs are typically only needed when done by a business user and OData only allows single inserts/ updates.

best,
sebastian

[yelisap]

Hello Sebastian,

Thanks for your explanations.

Our use case is a business user upload an Excel which contains many records. And based on your answer auditlog plugin is not supported, we will need publish audit log event in our processes.

BTW, is there any plan to support this scenario?

Best Regards,
Ye Li

[sjvans]

hi @yelisap

i took it as a feature request, but no immediate plans.

best,
sebastian

[yelisap]

Hi Sebastian,
Thanks for the help on this 👍

Best Regards,
Ye Li

[maravissiv]

Hello Sebastian,
Will it be possible to use the plugin , if i prepare the payload myself for a bulk upload scenario and provide the array of auditlog payloads and emit that event.

https://github.com/cap-js/audit-logging/blob/main/srv/log2restv2.js#L115C2-L116C44
I see that the event handler in the library can handle array of records, and it will trigger parallel writes for each of the audit log via individual promises.
This should still be okay rite ?

Thanks,
Sivaram

[sjvans]

hi @maravissiv

it should work yes, but it's not an official feature. hence, you should probably do the Promise.all() yourself.

best,
sebastian

[maravissiv]

Thanks Sebastian, will try that way.