-----------------------------------------------------------
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

Reply via email to