*** eranrom has joined #openstack-storlets | 00:27 | |
*** eranrom has quit IRC | 00:31 | |
openstackgerrit | Takashi Kajinami proposed openstack/storlets: WIP: Revive zero-copy about GET execution on object-server https://review.openstack.org/327244 | 01:11 |
---|---|---|
*** eranrom has joined #openstack-storlets | 01:21 | |
*** eranrom has quit IRC | 01:26 | |
kota_ | hello, takashi | 02:06 |
kota_ | it looks like a bunch of conversations in the yesterday meeting. great. | 02:06 |
kota_ | takashi: I'm wondering we could consider to use splice that swift can also do zero copy GET from fd into network socket. | 02:08 |
kota_ | the reference is 7d0e5ebe690bf3cf41ccd970281d532a23284e58 | 02:08 |
kota_ | ah, https://github.com/openstack/swift/commit/7d0e5ebe690bf3cf41ccd970281d532a23284e58 | 02:09 |
kota_ | Sam worked on that for enabling it avoiding the copy from kernel space to user space when GET object requested. | 02:10 |
kota_ | it might be helpful you understand how it works. | 02:10 |
*** eranrom has joined #openstack-storlets | 02:16 | |
*** eranrom has quit IRC | 02:20 | |
openstackgerrit | Takashi Kajinami proposed openstack/storlets: Revive zero-copy about GET execution on object-server https://review.openstack.org/327244 | 02:34 |
*** takashi has joined #openstack-storlets | 02:37 | |
takashi | eranrom: I submitted a patch 327244 to revive zero copy in get. plz have a look if you have time | 02:44 |
patchbot | takashi: https://review.openstack.org/#/c/327244/ - storlets - Revive zero-copy about GET execution on object-server | 02:44 |
takashi | that patch depends on some existing refactoring. If you mind removing zero copy temporaly when we only merge dependencies, I'm going to squash them into one. | 02:46 |
takashi | the following is the dependency chain for protocol refactoring. (Base) patch 314140 -> patch 325553 -> patch 324663 -> patch 327244 | 02:49 |
patchbot | takashi: https://review.openstack.org/#/c/314140/ - storlets - Refactor Storlet protocol classes | 02:49 |
patchbot | takashi: https://review.openstack.org/#/c/325553/ - storlets - Improve handling of fds in StorletInvocationProtocol | 02:49 |
patchbot | takashi: https://review.openstack.org/#/c/324663/ - storlets - Merge StorletRequest classes | 02:49 |
patchbot | takashi: https://review.openstack.org/#/c/327244/ - storlets - Revive zero-copy about GET execution on object-server | 02:49 |
takashi | kota_: It's very cool patch. I think we can use that idea to improve data transfer, especially when we send data processed by storlet to web socket. I'll have a deep look later. :-) | 02:55 |
*** eranrom has joined #openstack-storlets | 03:10 | |
*** eranrom has quit IRC | 03:15 | |
*** eranrom has joined #openstack-storlets | 04:05 | |
*** eranrom has quit IRC | 04:09 | |
*** eranrom has joined #openstack-storlets | 04:19 | |
*** eranrom has quit IRC | 05:01 | |
*** eranrom has joined #openstack-storlets | 05:34 | |
*** eranrom has quit IRC | 06:00 | |
*** eranrom has joined #openstack-storlets | 06:18 | |
eranrom | takashi: I am proceeding with your suggested plan. | 06:45 |
eranrom | Now merging 314140 | 06:45 |
eranrom | https://review.openstack.org/#/c/314140/9 | 06:45 |
patchbot | eranrom: patch 314140 - storlets - Refactor Storlet protocol classes | 06:45 |
eranrom | will continues with patch 325553 -> patch 324663 -> patch 327244 | 06:46 |
patchbot | eranrom: https://review.openstack.org/#/c/325553/ - storlets - Improve handling of fds in StorletInvocationProtocol | 06:46 |
patchbot | eranrom: https://review.openstack.org/#/c/324663/ - storlets - Merge StorletRequest classes | 06:46 |
patchbot | eranrom: https://review.openstack.org/#/c/327244/ - storlets - Revive zero-copy about GET execution on object-server | 06:46 |
openstackgerrit | Merged openstack/storlets: Refactor Storlet protocol classes https://review.openstack.org/314140 | 07:02 |
*** eranrom has quit IRC | 07:32 | |
openstackgerrit | Merged openstack/storlets: Improve handling of fds in StorletInvocationProtocol https://review.openstack.org/325553 | 07:40 |
openstackgerrit | Merged openstack/storlets: Merge StorletRequest classes https://review.openstack.org/324663 | 07:44 |
*** eranrom has joined #openstack-storlets | 07:50 | |
*** openstackgerrit has quit IRC | 08:03 | |
*** openstackgerrit has joined #openstack-storlets | 08:03 | |
*** eranrom has quit IRC | 08:15 | |
*** eranrom has joined #openstack-storlets | 08:26 | |
*** eranrom has quit IRC | 08:27 | |
*** eranrom has joined #openstack-storlets | 08:34 | |
*** eranrom has quit IRC | 08:35 | |
*** eranrom has joined #openstack-storlets | 08:40 | |
takashi | eranrom: thank you for your lot of review! now we only have patch 327244 remaining. | 08:58 |
patchbot | takashi: https://review.openstack.org/#/c/327244/ - storlets - Revive zero-copy about GET execution on object-server | 08:58 |
eranrom | yes, I will look at it shortly. | 08:58 |
*** takashi has quit IRC | 09:20 | |
*** eranrom has quit IRC | 10:12 | |
*** eranrom has joined #openstack-storlets | 11:59 | |
*** eranrom has quit IRC | 12:04 | |
*** eranrom has joined #openstack-storlets | 12:54 | |
*** eranrom has quit IRC | 12:58 | |
*** eranrom has joined #openstack-storlets | 13:48 | |
*** eranrom has quit IRC | 14:01 | |
*** eranrom has joined #openstack-storlets | 14:04 | |
openstackgerrit | Takashi Kajinami proposed openstack/storlets: Remove unnecessory parameters from DockerStorletRequest https://review.openstack.org/325554 | 16:05 |
*** eranrom has quit IRC | 16:36 | |
*** eranrom has joined #openstack-storlets | 17:29 | |
*** eranrom has quit IRC | 17:34 | |
*** eranrom has joined #openstack-storlets | 17:52 | |
*** eranrom has quit IRC | 18:24 | |
*** eranrom has joined #openstack-storlets | 18:49 | |
*** eranrom has quit IRC | 19:17 | |
*** eranrom has joined #openstack-storlets | 19:46 | |
*** eranrom has quit IRC | 19:51 | |
*** eranrom has joined #openstack-storlets | 20:40 | |
*** eranrom has quit IRC | 20:45 | |
*** eranrom has joined #openstack-storlets | 21:35 | |
*** eranrom has quit IRC | 21:40 | |
*** eranrom has joined #openstack-storlets | 22:29 | |
*** eranrom has quit IRC | 22:34 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!