[GitHub] samza pull request #417: SAMZA-1489; TaskInstance should commit offset befor...

2018-02-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/417 ---

Re: No Metrics Emitted in new High-Level API

2018-02-01 Thread Yi Pan
Hi, Srujun, Sorry to reply late. I am glad that you figured it out. The issue you reported seems to be a bug that we set the default for metrics serde in one case and not in the other one. I will open a ticket to track the investigation. Thanks a lot for reporting the issue! -Yi On Tue, Jan 30,

Re: No Metrics Emitted in new High-Level API

2018-02-01 Thread Yi Pan
SAMZA-1581 is the ticket On Thu, Feb 1, 2018 at 5:20 PM, Yi Pan wrote: > Hi, Srujun, > > Sorry to reply late. I am glad that you figured it out. The issue you > reported seems to be a bug that we set the default for metrics serde in one > case and not in the other one. I will open a ticket to tr

Re: [DISCUSS] SEP: Host affinity in standalone.

2018-02-01 Thread Yi Pan
Hi, Santhoosh, Thanks for the SEP and the latest revisions. Here are some of my comments based on the latest proposal: - The basic idea for implementing state-aware task-to-physical-process assignment in JobModel is not quite clear. ContainerAllocator is solving a different problem in host-affini

Re: SEP-11. Host Affinity in standalone discussion.

2018-02-01 Thread Yi Pan
Linking Boris' earlier comment in another email to this thread: http://mail-archives.apache.org/mod_mbox/samza-dev/201801.mbox/%3CCAPAaT%2BtH2H5TEvFQUn9jw6iR%3DyvVEu46rDLJsqexpwKz0CAH1g%40mail.gmail.com%3E On Fri, Jan 26, 2018 at 4:17 PM, Boris S wrote: > Shanthoosh, > Thank you for suggesting a

Re: [DISCUSS] SEP: Host affinity in standalone.

2018-02-01 Thread Yi Pan
Linking Boris' earlier comments to the correct [DISCUSSION] thread: http://mail-archives.apache.org/mod_mbox/samza-dev/201801.mbox/%3CCAPAaT%2BtH2H5TEvFQUn9jw6iR%3DyvVEu46rDLJsqexpwKz0CAH1g%40mail.gmail.com%3E On Thu, Feb 1, 2018 at 5:27 PM, Yi Pan wrote: > Hi, Santhoosh, > > Thanks for the SEP