Full_Name: Howard Chu Version: 2.5 OS: URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (83.136.45.114) Submitted by: hyc Some small frontend tweaks to better support back-asyncmeta
in master
changed notes changed state Open to Test moved from Incoming to Software Enhancements
As part of asyncmeta development, the front-end was made aware if operations were executed by an async back-end and cleanup is to be performed by the backend. This patch adds the same exception of extended operations, which were skipped because asyncmeta did not handle them. ftp://ftp.openldap.org/incoming/nadezhda-ivanova-180425.patch
This seems to be missing the following necessary change: diff --git a/servers/slapd/extended.c b/servers/slapd/extended.c index 2b55438..59624f7 100644 --- a/servers/slapd/extended.c +++ b/servers/slapd/extended.c @@ -176,6 +176,12 @@ do_extended( op->o_bd = frontendDB; rs->sr_err = frontendDB->be_extended( op, rs ); + if ( rs->sr_err == SLAPD_ASYNCOP ){ + /* skip cleanup */ + Debug( LDAP_DEBUG_TRACE, "do_extended: skipping cleanup for ASYNCOP\n", 0, 0, 0 ); + return rs->sr_err; + } + /* clean up in case some overlay set them? */ if ( !BER_BVISNULL( &op->o_req_ndn ) ) { if ( !BER_BVISNULL( &op->o_req_dn )
Note: Adjust debug statement there for head/RE25. ;)
Sigh, wrong commit in the message, but: Commits: • 8d312196 by Quanah Gibson-Mount at 2020-08-26T21:55:39+00:00 More for ITS#8845, skip cleanup on async op with extended operations