[
https://forge.continuent.org/jira/browse/SEQUOIA-1037?page=comments#action_14275
]
Gilles Rayrat commented on SEQUOIA-1037:
----------------------------------------
Current code in SelectRequest:
public boolean needsMacroProcessing()
{
return true;
}
Suggested fix:
public boolean needsMacroProcessing()
{
// replace macros only if the select does update(s)
return mustBroadcast;
}
> Macro replacement should not occur for select requests
> ------------------------------------------------------
>
> Key: SEQUOIA-1037
> URL: https://forge.continuent.org/jira/browse/SEQUOIA-1037
> Project: Sequoia
> Type: Improvement
> Components: Core
> Reporter: Gilles Rayrat
> Fix For: sequoia 2.10.10
>
>
> Macros like now() and rand() are currently replaced by static values in the
> controller.
> Unless the select request needs broadcast (eg. select for update), these
> macros should not be replaced in read-only requests as there won't be any
> side effect (inconsistent update across backends) as for updates.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://forge.continuent.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
_______________________________________________
Sequoia mailing list
[email protected]
https://forge.continuent.org/mailman/listinfo/sequoia
