searsia
/
searsiaclient
1
0
Fork 1
Code Issues 1 Pull Requests Packages Projects Releases 6 Wiki Activity

Return the session id directly #3

Merged
krisbreuker merged 1 commits from newapi into newapi 2019-04-27 19:26:46 +00:00
Conversation 1 Commits 1 Files Changed 1
krisbreuker commented 2019-04-26 14:41:43 +00:00 (Migrated from github.com)

If searchFederated is called twice quickly, the 'start' events for both searches may arrive out of order, so you still can't know which events belong to the last call to searchFederated. By returning the session id directly from searchFederated, we can ignore incoming events having a different (old) session id.

If searchFederated is called twice quickly, the 'start' events for both searches may arrive out of order, so you still can't know which events belong to the last call to searchFederated. By returning the session id directly from searchFederated, we can ignore incoming events having a different (old) session id.
djoerd commented 2019-04-27 19:25:08 +00:00 (Migrated from github.com)

You're right, and this is an elegant solution. Thanks. I will merge this.

You're right, and this is an elegant solution. Thanks. I will merge this.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
   
duplicate
   
enhancement
   
help wanted
   
invalid
   
question
   
wontfix
No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: searsia/searsiaclient#3
There is no content yet.
Delete Branch "newapi"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?