Skip to content

[ZP-9] Add support for variables in JDBCInterpreter statementPrecode.#4

Merged
aka-demik merged 1 commit intoZP-7from
BI-6300
Nov 1, 2018
Merged

[ZP-9] Add support for variables in JDBCInterpreter statementPrecode.#4
aka-demik merged 1 commit intoZP-7from
BI-6300

Conversation

@aka-demik
Copy link

@aka-demik aka-demik commented Oct 31, 2018

What is this PR for?

Variables #{noteId} and #{user} replaced to actual values.
Cherry picked from commits 9039f16, 4b90e87.

What type of PR is it?

Feature

Todos

What is the Jira issue?

ZP-9

How should this be tested?

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? yes

@aka-demik aka-demik force-pushed the BI-6300 branch 2 times, most recently from 8fe3865 to 3cff5d2 Compare October 31, 2018 14:32
Variables #{noteId} and #{user} replaced to actual values.
Cherry picked from commits 9039f16, 4b90e87
@aka-demik aka-demik changed the title [BI-6300] Add support for variables in JDBCInterpreter statementPrecode. [ZP-9] Add support for variables in JDBCInterpreter statementPrecode. Nov 1, 2018
@aka-demik aka-demik added the Tinkoff private When it makes no sense to share it with the community. label Nov 1, 2018
@aka-demik aka-demik changed the base branch from master to ZP-7 November 1, 2018 11:35
@aka-demik aka-demik merged commit e595f3e into ZP-7 Nov 1, 2018
@aka-demik aka-demik added Tinkoff private When it makes no sense to share it with the community. and removed Tinkoff private When it makes no sense to share it with the community. labels Nov 1, 2018
@aka-demik aka-demik deleted the BI-6300 branch November 2, 2018 07:13
@aka-demik aka-demik added this to the ZP-7 sync with upstream milestone Nov 23, 2018
egorklimov pushed a commit that referenced this pull request Jul 30, 2019
…t_limit to V_1.0.0

* commit '5dd5f7a68ccb32f1a4560e7015d8c51b806f16d5':
  [ZP-358] Fix limit on resultSet realisation
  Fix query result row limit
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Tinkoff private When it makes no sense to share it with the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants