You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@samza.apache.org by Xinyu Liu <xi...@gmail.com> on 2016/08/19 21:28:26 UTC
Review Request 51252: SAMZA-1004: Fix some logging and javadoc issues
for AsyncStreamTask
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/
-----------------------------------------------------------
Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
Repository: samza
Description
-------
SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
Diffs
-----
samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
Diff: https://reviews.apache.org/r/51252/diff/
Testing
-------
Thanks,
Xinyu Liu
Re: Review Request 51252: SAMZA-1004: Fix some logging and javadoc
issues for AsyncStreamTask
Posted by Boris Shkolnik <bo...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/#review146224
-----------------------------------------------------------
Ship it!
- Boris Shkolnik
On Aug. 19, 2016, 9:35 p.m., Xinyu Liu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51252/
> -----------------------------------------------------------
>
> (Updated Aug. 19, 2016, 9:35 p.m.)
>
>
> Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
>
>
> Repository: samza
>
>
> Description
> -------
>
> SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
>
>
> Diffs
> -----
>
> samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
> samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
>
> Diff: https://reviews.apache.org/r/51252/diff/
>
>
> Testing
> -------
>
> gradlew clean build.
>
>
> Thanks,
>
> Xinyu Liu
>
>
Re: Review Request 51252: SAMZA-1004: Fix some logging and javadoc
issues for AsyncStreamTask
Posted by Navina Ramesh <nr...@linkedin.com>.
> On Aug. 30, 2016, 10:37 p.m., Prateek Maheshwari wrote:
> > samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java, line 36
> > <https://reviews.apache.org/r/51252/diff/2/?file=1488893#file1488893line36>
> >
> > s/serialized/serial?
+1
- Navina
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/#review147373
-----------------------------------------------------------
On Aug. 30, 2016, 10:01 p.m., Xinyu Liu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51252/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2016, 10:01 p.m.)
>
>
> Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
>
>
> Repository: samza
>
>
> Description
> -------
>
> SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
>
>
> Diffs
> -----
>
> samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
> samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
>
> Diff: https://reviews.apache.org/r/51252/diff/
>
>
> Testing
> -------
>
> gradlew clean build.
>
>
> Thanks,
>
> Xinyu Liu
>
>
Re: Review Request 51252: SAMZA-1004: Fix some logging and javadoc
issues for AsyncStreamTask
Posted by "Yi Pan (Data Infrastructure)" <yi...@linkedin.com>.
> On Aug. 30, 2016, 10:37 p.m., Prateek Maheshwari wrote:
> > samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java, line 25
> > <https://reviews.apache.org/r/51252/diff/2/?file=1488893#file1488893line25>
> >
> > multithreaded.
> >
> > Maybe use asynchronous instead of multithreaded here? Multi-threaded seems like an implementation detail.
Maybe "asynchronous execution in a multi-threaded Samza container" is more accurate?
- Yi
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/#review147373
-----------------------------------------------------------
On Aug. 30, 2016, 10:01 p.m., Xinyu Liu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51252/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2016, 10:01 p.m.)
>
>
> Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
>
>
> Repository: samza
>
>
> Description
> -------
>
> SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
>
>
> Diffs
> -----
>
> samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
> samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
>
> Diff: https://reviews.apache.org/r/51252/diff/
>
>
> Testing
> -------
>
> gradlew clean build.
>
>
> Thanks,
>
> Xinyu Liu
>
>
Re: Review Request 51252: SAMZA-1004: Fix some logging and javadoc
issues for AsyncStreamTask
Posted by Prateek Maheshwari <pm...@linkedin.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/#review147373
-----------------------------------------------------------
samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java (line 25)
<https://reviews.apache.org/r/51252/#comment214492>
multithreaded.
Maybe use asynchronous instead of multithreaded here? Multi-threaded seems like an implementation detail.
samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java (line 36)
<https://reviews.apache.org/r/51252/#comment214493>
s/serialized/serial?
- Prateek Maheshwari
On Aug. 30, 2016, 3:01 p.m., Xinyu Liu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51252/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2016, 3:01 p.m.)
>
>
> Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
>
>
> Repository: samza
>
>
> Description
> -------
>
> SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
>
>
> Diffs
> -----
>
> samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
> samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
>
> Diff: https://reviews.apache.org/r/51252/diff/
>
>
> Testing
> -------
>
> gradlew clean build.
>
>
> Thanks,
>
> Xinyu Liu
>
>
Re: Review Request 51252: SAMZA-1004: Fix some logging and javadoc
issues for AsyncStreamTask
Posted by Prateek Maheshwari <pm...@linkedin.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/#review147583
-----------------------------------------------------------
Ship it!
While we're on the topic of logging: I found it useful to have a trace log for when the AsyncRunLoop gets unblocked in blockIfBusy. We already have one for when it blocks.
- Prateek Maheshwari
On Aug. 30, 2016, 3:01 p.m., Xinyu Liu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51252/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2016, 3:01 p.m.)
>
>
> Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
>
>
> Repository: samza
>
>
> Description
> -------
>
> SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
>
>
> Diffs
> -----
>
> samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
> samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
>
> Diff: https://reviews.apache.org/r/51252/diff/
>
>
> Testing
> -------
>
> gradlew clean build.
>
>
> Thanks,
>
> Xinyu Liu
>
>
Re: Review Request 51252: SAMZA-1004: Fix some logging and javadoc
issues for AsyncStreamTask
Posted by "Yi Pan (Data Infrastructure)" <yi...@linkedin.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/#review147582
-----------------------------------------------------------
Ship it!
Ship It!
- Yi Pan (Data Infrastructure)
On Aug. 30, 2016, 10:01 p.m., Xinyu Liu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51252/
> -----------------------------------------------------------
>
> (Updated Aug. 30, 2016, 10:01 p.m.)
>
>
> Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
>
>
> Repository: samza
>
>
> Description
> -------
>
> SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
>
>
> Diffs
> -----
>
> samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
> samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
>
> Diff: https://reviews.apache.org/r/51252/diff/
>
>
> Testing
> -------
>
> gradlew clean build.
>
>
> Thanks,
>
> Xinyu Liu
>
>
Re: Review Request 51252: SAMZA-1004: Fix some logging and javadoc
issues for AsyncStreamTask
Posted by Xinyu Liu <xi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/
-----------------------------------------------------------
(Updated Aug. 30, 2016, 10:01 p.m.)
Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
Repository: samza
Description
-------
SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
Diffs (updated)
-----
samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
Diff: https://reviews.apache.org/r/51252/diff/
Testing
-------
gradlew clean build.
Thanks,
Xinyu Liu
Re: Review Request 51252: SAMZA-1004: Fix some logging and javadoc
issues for AsyncStreamTask
Posted by Xinyu Liu <xi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51252/
-----------------------------------------------------------
(Updated Aug. 19, 2016, 9:35 p.m.)
Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).
Repository: samza
Description
-------
SAMZA-1004: Fix some logging and javadoc issues for AsyncStreamTask
Diffs
-----
samza-api/src/main/java/org/apache/samza/task/AsyncStreamTask.java 684ba0b77fca1c1c5fd8d740597d3b7fcdc0f5cb
samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala f786fc08c8f7eced4f4084dc8326b288888b6422
Diff: https://reviews.apache.org/r/51252/diff/
Testing (updated)
-------
gradlew clean build.
Thanks,
Xinyu Liu