remove __del__ because it's evil and also prevents the ResourceWarning on the socket from happening (closes #16900)

The fix for issue 9750 introduced a regression by sorting the row objects
returned by fetchall.  But if a row_factory such as sqlite3.Row is used, the
rows may not be sortable (in Python3), which leads to an exception.  The
sorting is still a nice idea, so the patch moves the sort into the sql.

Fix and test by Peter Otten.

The fix for issue 9750 introduced a regression by sorting the row objects
returned by fetchall.  But if a row_factory such as sqlite3.Row is used, the
rows may not be sortable (in Python3), which leads to an exception.  The
sorting is still a nice idea, so the patch moves the sort into the sql.

Fix and test by Peter Otten.

The fix for issue 9750 introduced a regression by sorting the row objects
returned by fetchall.  But if a row_factory such as sqlite3.Row is used, the
rows may not be sortable (in Python3), which leads to an exception.  The
sorting is still a nice idea, so the patch moves the sort into the sql.

Fix and test by Peter Otten.

Patch by Serhiy Storchaka

Patch by Serhiy Storchaka

with the Python version of the class.

Patch by Daniel Shahaf.

with the Python version of the class.

Patch by Daniel Shahaf.

#16910: test_bytes, test_unicode, and test_userstring now work with unittest test discovery.  Patch by Zachary Ware.

#16852: test_genericpath, test_posixpath, test_ntpath, and test_macpath now work with unittest test discovery.  Patch by Zachary Ware.

buffer instead of allocating a new one at each poll().