Este [site][1] (Matheus Piscioneri) resumiu muito bem os motivos.

 - É mais difícil que escrever seu próprio TableModel *(fato)*;
 - É mais lento (usa classes sincronizadas) (se olhar o código verá que a classe utiliza `Vector` e não `List`);
 - Ocupa mais espaço em memória (duplica seus dados) *(Ainda não consegui testar)* ;
 - Deixa o código mais confuso e difícil de manter *(fato)*;
 - Usa casts inseguros (por exemplo no método `convertToVector(Object[] anArray)` onde : `Vector<Object> v = new Vector<Object>(anArray.length);`

        for (Object o : anArray) {
            v.addElement(o);
        };

 - Força que você tenha que exibir informações desnecessárias (como o ID) na  tabela, ou controlar o ID numa lista separada;
 - *Faz a sua mulher te deixar, o leite da sua geladeira azedar, e pessoas apontarem o dedo para você na rua.(Isso realmente aconteceu);*

Uma prática que **ViniGodoy** sempre recomendou.

  [1]: http://matheuspiscioneri.com.br/blog/nao-use-defaulttablemodel-e-sim-abstracttablemodel/