hcq/MaxScale
hcq/MaxScale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Combine and rename session statement functions

Browse Source 
There was no real need to have two separate functions for getting the
stored buffer and the target server. Combining them into one allows it to
be handled in a nicer way.
This commit is contained in:
Markus Makela 2016-12-07 13:03:54 +02:00
parent 85d879e6ae
commit d42ef8ed13
2 changed files with 16 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
include/maxscale/session.h
View File

@ -395,18 +395,11 @@ bool session_store_stmt(SESSION *session, GWBUF *buf, const struct server *serve
* The value returned by this call must be freed by the caller with gwbuf_free().
*
* @param session Session with a stored statement
* @return Stored statement or NULL if no statement was stored
* @param buffer Pointer where the buffer is stored
* @param target Pointer where target server is stored
* @return True if a statement was stored
*/
GWBUF* session_fetch_stmt(SESSION *session);
/**
* Fetch the stored target
*
* @param session Session whose target is fetched
* @return The server where the statement is executed or NULL if no statement is
* stored
*/
const struct server* session_fetch_stmt_target(const SESSION *session);
bool session_take_stmt(SESSION *session, GWBUF **buffer, const struct server **target);
/**
* Clear the stored statement

20
server/core/session.c
View File

@ -964,16 +964,20 @@ bool session_store_stmt(SESSION *session, GWBUF *buf, const SERVER *server)
return rval;
}
GWBUF *session_fetch_stmt(SESSION *session)
bool session_take_stmt(SESSION *session, GWBUF **buffer, const SERVER **target)
{
GWBUF *rval = session->stmt.buffer;
session->stmt.buffer = NULL;
return rval;
}
bool rval = false;
const SERVER* session_fetch_stmt_target(const SESSION *session)
{
return session->stmt.target;
if (session->stmt.buffer && session->stmt.target)
{
*buffer = session->stmt.buffer;
*target = session->stmt.target;
session->stmt.buffer = NULL;
session->stmt.target = NULL;
rval = true;
}
return rval;
}
void session_clear_stmt(SESSION *session)