[issue30270] Remove sqlite3.Cache display method

2018-07-22 Thread Berker Peksag 


Berker Peksag  added the comment:

Closing this as 'rejected' for the reasons given by Brett and David. Thanks for 
working on this!

--
resolution:  -> rejected
stage:  -> 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

[issue30270] Remove sqlite3.Cache display method

2017-05-05 Thread Berker Peksag 

Berker Peksag added the comment:

FWIW, I agree with Brett and David. Did you try to use it? I prefer to fix it 
even if it's broken at the moment.

--

___
Python tracker 

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

[issue30270] Remove sqlite3.Cache display method

2017-05-05 Thread R. David Murray 

R. David Murray added the comment:

If it is there for debugging, why would we want to remove it?

--
nosy: +r.david.murray

___
Python tracker 

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

[issue30270] Remove sqlite3.Cache display method

2017-05-05 Thread Brett Cannon 

Brett Cannon added the comment:

With over 10 years of existence for that method, I can guarantee you someone 
relies on it, and so we should do a reasonable job to avoid breaking them 
without at least some warning first.

--

___
Python tracker 

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

[issue30270] Remove sqlite3.Cache display method

2017-05-05 Thread Aviv Palivoda 

Aviv Palivoda added the comment:

>From looking at the git blame this is there since 2006. Maybe this should be 
>dependent on issue #30262. Maybe at first there should be a deprecation 
>warning and then removing this later.
I understand that this might break someone's code but this is an undocumented 
method of an undocumented class in the sqlite3 module. I don't think that there 
will be a lot of people that use the sqlite3.Cache object.

--

___
Python tracker 

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

[issue30270] Remove sqlite3.Cache display method

2017-05-04 Thread Brett Cannon 

Brett Cannon added the comment:

How long has the method been there? And does it hurt anything for it to be 
exposed? Removing it at this point may break code if it predates Python 3.7.

--
nosy: +brett.cannon

___
Python tracker 

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

[issue30270] Remove sqlite3.Cache display method

2017-05-04 Thread Aviv Palivoda 

Changes by Aviv Palivoda :


--
pull_requests: +1559

___
Python tracker 

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

[issue30270] Remove sqlite3.Cache display method

2017-05-04 Thread Aviv Palivoda 

New submission from Aviv Palivoda:

The display method is there for debugging and should not be in the released 
code.

--
components: Extension Modules
messages: 293005
nosy: berker.peksag, ghaering, palaviv
priority: normal
severity: normal
status: open
title: Remove sqlite3.Cache display method
type: enhancement
versions: Python 3.7

___
Python tracker 

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