Method destroy

src/dsql/DsqlRequests.cpp:164–227

Release a dynamic request.

// Release a dynamic request.
void DsqlRequest::destroy(thread_db* tdbb, DsqlRequest* dsqlRequest)
{
	SET_TDBB(tdbb);

	if (dsqlRequest->req_timer)
	{
		dsqlRequest->req_timer->stop();
		dsqlRequest->req_timer = nullptr;
	}

	// If request is parent, orphan the children and release a portion of their requests

	for (auto childStatement : dsqlRequest->cursors)
	{
		childStatement->addFlags(DsqlStatement::FLAG_ORPHAN);
		childStatement->setParentRequest(nullptr);
		childStatement->setParentDbKey(nullptr);
		childStatement->setParentRecVersion(nullptr);
		dsqlRequest->req_dbb->dbb_statement_cache->removeStatement(tdbb, childStatement);

		// hvlad: lines below is commented out as
		// - child is already unlinked from its parent request
		// - we should not free child's sql text until its owner request is alive
		// It seems to me we should destroy owner request here, not a child
		// statement - as it always was before

		//Jrd::ContextPoolHolder context(tdbb, &childStatement->getPool());
		//releaseStatement(childStatement);
	}

	// If the request had an open cursor, close it

	if (dsqlRequest->req_cursor)
		DsqlCursor::close(tdbb, dsqlRequest->req_cursor);

	if (dsqlRequest->req_batch)
	{
		delete dsqlRequest->req_batch;
		dsqlRequest->req_batch = nullptr;
	}

	Jrd::Attachment* att = dsqlRequest->req_dbb->dbb_attachment;
	const bool need_trace_free = dsqlRequest->req_traced && TraceManager::need_dsql_free(att);
	if (need_trace_free)
	{
		TraceSQLStatementImpl stmt(dsqlRequest, NULL);
		TraceManager::event_dsql_free(att, &stmt, DSQL_drop);
	}

	if (dsqlRequest->req_cursor_name.hasData())
		dsqlRequest->req_dbb->dbb_cursors.remove(dsqlRequest->req_cursor_name);

	// If a request has been compiled, release it now
	if (dsqlRequest->getRequest())
		EXE_release(tdbb, dsqlRequest->getRequest());

	// Increase the statement refCount so its pool is not destroyed before the request is gone.
	auto dsqlStatement = dsqlRequest->getDsqlStatement();