Issue 7290 in pharo: Add #split: streaming protocol

Status: Accepted
Owner: [hidden email]
Labels: Type-Bug Target-Collection

New issue 7290 by [hidden email]: Add #split: streaming protocol
http://code.google.com/p/pharo/issues/detail?id=7290

Currently the split: message always creates an intermediate collection.
In certain cases it would be nice to have a streaming protocol for this to  
avoid the substantial overhead of creating intermediate OrderedCollections.

Suggestion:
===========
- add #split:indexesDo: which streams the split positions
- add #split:do: which streams the single splitted subcollections
- rewrite #split: to use #split:do:



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Labels: -Type-Bug Type-Enh

Comment #1 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixReviewNeeded

Comment #2 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290

Slice in Inbox:

added streaming protocol for #split:
- new method #split:do:
- new method #split:indicesDo:
- recategorized all split/join methods into *Collections-Abstract-splitjoin  
resulting in less polution


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Labels: Milestone-2.0

Comment #3 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #4 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290

Camillo this is strange when I merge the version two I get an empty change.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #5 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290

checking...


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #6 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290

Name: SLICE-Issue-7290-Add-split-streaming-protocol-CamilloBruni.3
Author: CamilloBruni
Time: 28 January 2013, 1:26:05.542 pm
UUID: 9f79a771-cdd8-4988-82a8-a9f1eed6fbf2
Ancestors: SLICE-Issue-7290-Add-split-streaming-protocol-CamilloBruni.1
Dependencies: Kernel-CamilloBruni.1290, Regex-Core-CamilloBruni.21,  
Collections-Abstract-CamilloBruni.195,  
Collections-Sequenceable-CamilloBruni.125,  
Collections-Strings-CamilloBruni.239

saving again, seems like the intermediate update-merge had a strange  
side-effect on my added methods


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: MonkeyIsChecking

Comment #7 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290#c7

The Monkey is currently checking this issue. Please don't change it!


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: ValidatedByTheMonkey
        Labels: CheckedIn20493

Comment #8 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290#c8

This Issue has been checked by Ulysse the Monkey
       
7630 tests passed in 00:06:05s:
===============================
        CollectionsTests-Arrayed (553)
        CollectionsTests-Atomic (12)
        CollectionsTests-Sequenceable (912)
        CollectionsTests-SplitJoin (27)
        CollectionsTests-Stack (16)
        CollectionsTests-Streams (37)
        CollectionsTests-Strings (611)
        CollectionsTests-Support (12)
        CollectionsTests-Unordered (1954)
        CollectionsTests-Weak (739)
        CompilerTests (181)
        KernelTests-Chronology (1186)
        KernelTests-Classes (136)
        KernelTests-Exception (5)
        KernelTests-Methods (359)
        KernelTests-Numbers (552)
        KernelTests-Objects (172)
        KernelTests-Pragmas (6)
        KernelTests-Processes (76)
        SUnit-Core-Extensions (3)
        SUnit-Core-Utilities (3)
        SUnit-Tests-Core (78)

----------------------------------------------------------
Loaded Source: SLICE-Issue-7290-Add-split-streaming-protocol-CamilloBruni.3  
from http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20493-a on CoInterpreter  
VMMaker-oscog-EstebanLorenzano.166 uuid:  
5773fcb9-2982-4507-8a9e-4308ec33731e Dec 12 2012
StackToRegisterMappingCogit VMMaker-oscog-EstebanLorenzano.166 uuid:  
5773fcb9-2982-4507-8a9e-4308ec33731e Dec 12 2012
git://gitorious.org/cogvm/blessed.git Commit:  
452863bdfba2ba0b188e7b172e9bc597a2caa928 Date: 2012-12-07 16:49:46 +0100  
By: Esteban Lorenzano <[hidden email]> Jenkins build #5922


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixToInclude

Comment #9 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #10 on issue 7290 by [hidden email]: Add #split: streaming  
protocol
http://code.google.com/p/pharo/issues/detail?id=7290

in 2.0 494


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker