mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anand Mazumdar <an...@apache.org>
Subject Re: Review Request 61579: Ensured JAVA HTTP adapter propagates a subscription error.
Date Wed, 30 Aug 2017 21:38:08 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61579/#review184203
-----------------------------------------------------------


Fix it, then Ship it!




LGTM minus a small nit.


src/java/jni/org_apache_mesos_v1_scheduler_V0Mesos.cpp
Lines 534-537 (patched)
<https://reviews.apache.org/r/61579/#comment260272>

    This is not neccessarily true. An error can also be due to other things like providing
the wrong master URL etc.
    
    Can we clarify here that we do this workaround to mimic the invariant of the v1 interface
that we can only receive an event after successfully connecting with the master?


- Anand Mazumdar


On Aug. 30, 2017, 2:16 p.m., Alexander Rukletsov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61579/
> -----------------------------------------------------------
> 
> (Updated Aug. 30, 2017, 2:16 p.m.)
> 
> 
> Review request for mesos, Anand Mazumdar, Till Toenshoff, and Vinod Kone.
> 
> 
> Bugs: MESOS-7872
>     https://issues.apache.org/jira/browse/MESOS-7872
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Prior to this patch, if an error occurred during subscription /
> registration to the master, it was not propagated back to the
> scheduler if the HTTP adapter was used. This happened because
> the HTTP adapter does not call `scheduler.connected` until after
> successful registration and hence the scheduler does not try to
> send the `SUBSCRIBE` call, without which the adapter does not
> send any events to the scheduler.
> 
> A fix is to call `scheduler.connected` if an error occurred
> before the scheduler had subscribed.
> 
> 
> Diffs
> -----
> 
>   src/java/jni/org_apache_mesos_v1_scheduler_V0Mesos.cpp 1f58fbff4e8414e4d2ae4c8f69b637ee3315e411

> 
> 
> Diff: https://reviews.apache.org/r/61579/diff/2/
> 
> 
> Testing
> -------
> 
> See https://reviews.apache.org/r/61580/
> 
> 
> Thanks,
> 
> Alexander Rukletsov
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message