Exchange Stream API Release - 19th Nov 2019 - SUB_IMAGE change

Collapse
This topic is closed.
X
X
 
  • Time
  • Show
Clear All
new posts

  • BetfairDeveloperProgram
    replied
    Thanks LiamP!!

    Neil

    Leave a comment:


  • LiamP
    replied
    No issues for betfairlightweight, it just replaces the cache with the new image, see attached for an example market with a sub image in the middle that others may find useful.
    Attached Files

    Leave a comment:


  • LiamP
    replied
    Excellent, thanks Neil.

    Leave a comment:


  • LetsGo
    replied
    Good sense and sound judgement in practical matters as always from the BDP team.

    Leave a comment:


  • BetfairDeveloperProgram
    replied
    The above release has now been delayed until the 19th November and new testing times have been added from Monday - Friday next week.

    Kind Regards

    Neil

    Leave a comment:


  • BetfairDeveloperProgram
    replied
    Morning Liam.

    Yes, can add an additional test period on on Monday moring at 10am. I'll update the schedule accordingly.

    I'll know whether or not the release on Tuesday will go ahead as planned before the of the day and will provide an update regarding this then,

    Kind Regards

    Neil

    Leave a comment:


  • LiamP
    replied
    Originally posted by BetfairDeveloperProgram View Post
    Today's test on the stream-api-integration.betfair.com endpoint worked for some connections but the image executor got overloaded and had to disconnect the remaining ones.

    We are preparing a fix for the next test which will now take place on Friday at 10:00 GMT.

    Kind Regards

    Neil
    Hi Neil,

    I am not going to be available on Friday, is it possible to add a test next week as well? Seem to be cutting it very fine with it being released on Tuesday, will this be pushed back due to today’s problems?

    Leave a comment:


  • BetfairDeveloperProgram
    replied
    Today's test on the stream-api-integration.betfair.com endpoint worked for some connections but the image executor got overloaded and had to disconnect the remaining ones.

    We are preparing a fix for the next test which will now take place on Friday at 10:00 GMT.

    Kind Regards

    Neil
    Last edited by BetfairDeveloperProgram; 06-11-2019, 05:05 PM.

    Leave a comment:


  • WTPooh
    replied
    It looks like no changes are needed for the sample C# client.
    Here is how it works now:
    MarketCache.cs

    public void OnMarketChange(ChangeMessage<MarketChange> changeMessage)
    {
    if (changeMessage.IsStartOfNewSubscription)
    {
    //clear cache
    _markets.Clear();
    }
    //...
    }

    ChangeMessage.cs

    public bool IsStartOfNewSubscription
    {
    get
    {
    return ChangeType == ChangeType.SUB_IMAGE &&
    (SegmentType == SegmentType.NONE || SegmentType == SegmentType.SEG_START);
    }
    }

    Leave a comment:


  • bfexplorer
    replied
    Do you really simulate this new behaviour because I just switched to stream-api-integration.betfair.com, and I did not notice SUB_IMAGE.

    I have done test on my desktop app, and as well prepared simple code for testing, so the question is how we should write our unit test code for this new feature?

    Well yes, all seems to be working as before for me.

    Here is video from my test:

    https://youtu.be/oNnA1NI_Xps

    Leave a comment:


  • LiamP
    replied
    Anyone have a connection that got data? Mine failed...


    Code:
    betfairlightweight.exceptions.ListenerError: 
    connection_id: 100-061119095447-34, 
    data: {"op":"status","statusCode":"FAILURE","errorCode":"UNEXPECTED_ERROR","errorMessage":"Unable to provide initial image due to capacity limitations, please try again","connectionClosed":true,"connectionId":"100-0611134395937-3434"}

    Leave a comment:


  • LetsGo
    replied
    Hi Neil,

    What are the consequence of not making this change to my code?
    For example, will I be unsubscribed to the market or just lose a streaming market change?

    Thanks

    Leave a comment:


  • geoffw123
    replied
    Hi Neil

    Presumbably this change means a code change is required in your Streaming example code ? If so can you confirm if and when you will be updating the github client example code ?

    Thanks

    Geoff

    Leave a comment:


  • BetfairDeveloperProgram
    replied
    Hi LiamP.

    i've now added details of when and how this can be tested in the announcement above.

    Thanks

    Neil

    Leave a comment:


  • BetfairDeveloperProgram
    replied
    Hi LiamP

    Yes, we'll add this capability and will let you know as soon as this is available.

    Kind Regards

    Neil

    Leave a comment:

Working...
X