[issue28340] [py2] TextIOWrapper.tell extremely slow

2017-11-06 Thread STINNER Victor 

STINNER Victor  added the comment:

Berker Peksag, Antoine Pitrou (who implemented the discussed change) and me are 
opposed to backporting this optimization, so I close the issue as WONTFIX.

The only known workaround is to upgrade to Python 3, sorry!

Benjamin: Please speakup if you want this optimization in the next Python 2.7 
release ;-)


> The original patch is quite delicate and I'm not comfortable backporting to 
> 2.7.

Oh, the "byte/character ratio" "hack"... I'm not confortable with this one 
neither, even if it seems like it works on Python 3 (I didn't recall any bug 
report related to this).

--
nosy: +haypo
resolution:  -> wont fix
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue28340] [py2] TextIOWrapper.tell extremely slow

2017-11-04 Thread Antoine Pitrou 

Antoine Pitrou  added the comment:

The original patch is quite delicate and I'm not comfortable backporting to 
2.7.  This is why it was pushed to 3.3 but not backported to 3.2.x (AFAIR).  
2.7 has existed with this issue for years, now, and is slowly nearing 
end-of-life.  I'll let Benjamin make the final decision.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue28340] [py2] TextIOWrapper.tell extremely slow

2017-11-04 Thread Berker Peksag 

Berker Peksag  added the comment:

>From the original report:

> This problem did not exist in version 2.7.1 [...]

We usually don't treat performance improvements as bug fixes unless it was 
introduced in a bug fix release (or it needs to be explicitly approved by the 
BDFL or the 2.7 release manager)

If TextIOWrapper.tell() performance got worse after 2.7.1, there might be a 
chance to backport issue 4 to 2.7.

Adding Antoine (author of the patch) and Benjamin (2.7. release manager) to get 
their opinion.

--
nosy: +benjamin.peterson, berker.peksag, pitrou

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue28340] [py2] TextIOWrapper.tell extremely slow

2017-11-04 Thread Mariatta Wijaya 

Mariatta Wijaya  added the comment:

Anyone here capable of reviewing the PR?

--
nosy: +Mariatta
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue28340] [py2] TextIOWrapper.tell extremely slow

2017-05-22 Thread Rob Malouf 

Changes by Rob Malouf :


--
pull_requests: +1832

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue28340] [py2] TextIOWrapper.tell extremely slow

2016-10-18 Thread STINNER Victor 

Changes by STINNER Victor :


--
title: TextIOWrapper.tell extremely slow -> [py2] TextIOWrapper.tell extremely 
slow

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com