#48769 RFE: Be_txn extended operation plugin type
Closed: wontfix None Opened 6 years ago by firstyear.

http://www.port389.org/docs/389ds/design/exop-plugin-transactions.html

https://fedorahosted.org/389/ticket/48342

Due to interactions with pre and post op be_txn plugins, we should have the option for extended operations to support a be_txn type of plugin also. This will be used by dna.c, and other plugins in the future. This helps to reduce developer stress on managing transactions, and will improve dirsrv stability. Please see the design document, and the trac ticket for more.


The framework implementation looks good.

Probably, this might be a style issue, but I prefer rather having this kind of notes in the ticket, not in the comments, which could be easily ignored and might be accidentally removed.

337     /* wibrown 201603 I want to rewrite this to get plugin p, and use that  
338      * rather than all these plugin_call_, that loop over the plugin lists 
339      * We do "get plugin (oid). 
340      * then we just hand *p into the call functions. 
341      * much more efficient! :) 
342      */

commit 0204d58427aa683a1999868b2952d2495277429f
commit da633b37db56ca63f2a9b22bd6feaa9d1c3a9f9b
Total 21 (delta 16), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
9b3c273..1d217fe master -> master

Metadata Update from @firstyear:
- Issue assigned to firstyear
- Issue set to the milestone: 1.3.5.0

5 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/1829

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: Fixed)

2 years ago

Login to comment on this ticket.

Metadata