-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37506/
-----------------------------------------------------------
Review request for samza, Yi Pan (Data Infrastructure) and Navina Ramesh.
Bugs: SAMZA-552
https://issues.apache.org/jira/browse/SAMZA-552
Repository: samza
Description
-------
New proposal for TopologuBuilder API proposed in rb34500
(https://reviews.apache.org/r/34500/).
* Created a new class called TopologyBuilderV2 instead of changing existing
TopologyBuilder
* org.apache.samza.sql.operators.factory.TestTopologyBuilderV2 contains two
tests which demonstrate the basic usage of the new API
* Window and aggregate related draft APIs are not done yet
* This is a WIP, please feel free to comment on the APIs
Diffs
-----
samza-sql-calcite/src/main/java/org/apache/samza/sql/calcite/schema/AvroSchemaConverter.java
705c0ffadd0982defae686bb0df178e9a7cbef8c
samza-sql-core/src/test/java/org/apache/samza/task/sql/RandomWindowOperatorTask.java
20dc70179e74070e8feb617b6a7b3b62ef5c1156
samza-sql-core/src/test/java/org/apache/samza/task/sql/StreamSqlTask.java
9124e3c7191aac8a738345c02b260baae811c5c5
samza-sql-core/src/test/java/org/apache/samza/task/sql/UserCallbacksSqlTask.java
96e96c363e723c20c30622d1253588f66166b206
Diff: https://reviews.apache.org/r/37506/diff/
Testing
-------
./gradlew :samza-sql-core:test passed
Thanks,
Milinda Pathirage