[
https://issues.apache.org/jira/browse/APEXMALHAR-2172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15399052#comment-15399052
]
ASF GitHub Bot commented on APEXMALHAR-2172:
--------------------------------------------
GitHub user DT-Priyanka opened a pull request:
https://github.com/apache/apex-malhar/pull/355
[review only]APEXMALHAR-2172: Updates to JDBC Poll Input Operator
Changes include:
1. Using jooq query to construct sql queries
2. Update logic to query based on row numbers instead of primary key column
values
3. Small fixes
4. Code refactoring to make it more readable
5. Update to test cases, to abstract out generic test code and add more
tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/DT-Priyanka/incubator-apex-malhar
APEXMALHAR-2172-jdbc-poller-input
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/apex-malhar/pull/355.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #355
----
commit 47a692c7382795f88e4ce7a2ca640b70b2aa0094
Author: Priyanka Gugale <[email protected]>
Date: 2016-07-19T06:40:08Z
APEXMALHAR-2172: Updates to JDBC Poll Input Operator
----
> Update JDBC poll input operator to fix issues
> ---------------------------------------------
>
> Key: APEXMALHAR-2172
> URL: https://issues.apache.org/jira/browse/APEXMALHAR-2172
> Project: Apache Apex Malhar
> Issue Type: Improvement
> Reporter: Priyanka Gugale
> Assignee: Priyanka Gugale
>
> Update JDBCPollInputOperator to:
> 1. Fix small bugs
> 2. Use jooq query dsl library to construct sql queries
> 3. Make code more readable
> 4. Use row counts rather than key column values to partition reads
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
