Transducer of partition-by over take gives wrong answer

Description

Environment

None

Activity

Show:
Alex Miller
October 21, 2014, 7:16 PM
Alex Miller
October 21, 2014, 5:09 PM

Rich has a (different) patch for this on the way.

Nicola Mometto
October 21, 2014, 4:03 PM

Patch 0001-CLJ-1571-fix-regression-introduced-by-43cc1854508d65.patch addresses this issue while preserving the current lazyness factor of `sequence`

Nicola Mometto
October 21, 2014, 3:53 PM

Applying this patch causes a regression in the lazyiness of sequence.
The lines that Ghadi removed for this patch were added by Rich in this commit https://github.com/clojure/clojure/commit/43cc1854508d655e58e377f84836ba128971f90c to address http://dev.clojure.org/jira/browse/CLJ-1497

Example of the regression:
current master:

with this patch:

Alex Miller
October 21, 2014, 3:41 PM

Thanks Ghadi. This bug was found by the tests I wrote for CLJ-1554, so yes.

Completed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Rich Hickey

Reporter

Alex Miller

Labels

Approval

Ok

Priority

Critical

Affects versions

Fix versions