Ticket #1751 (closed defeito: fixed)

Opened 8 years ago

Last modified 8 years ago

Lentidão na abertura da caixa dentro do ExpressoMail

Reported by: alexandrecorreia Owned by: alexandrecorreia
Priority: alta Milestone: Expresso 2.2.0-rc1
Component: ExpressoMail Version: branch 2.2
Severity: média Keywords: lentidão abertura caixa postal
Cc: WorkGroup:

Description

Esta lentidão foi ocasionada pela revisão 3929, isto afeta diretamente tb o módulo mobile.

Attachments

Linux.tar.bz2 Download (1.6 MB) - added by alexandrecorreia 8 years ago.
Mensagens para teste, este arquivo tem aproximadamente 1500 mensagens.

Change History

comment:1 follow-up: ↓ 3 Changed 8 years ago by alexandrecorreia

Revisão [4014] - Esta correção ainda está sendo testada por Roberto Santos Junior ( Prodeb ).

comment:2 Changed 8 years ago by rafaelraymundo

Favor verificar a codificação do seu ambiente de desenvolvimento, pois o arquivo alterado apresentou inconsistências na acentuação.

comment:3 in reply to: ↑ 1 Changed 8 years ago by eduardoalex

Replying to alexandrecorreia:

Revisão [4014] - Esta correção ainda está sendo testada por Roberto Santos Junior ( Prodeb ).

alexandre,

Reveja o a codificação desse arquivo, todos os acentos estão fora de codificação.

comment:4 Changed 8 years ago by roberto.santosjunior

Alexandre,

o problema da codificação so afetou uma função, que corigi aqui para adiantar os testes. Com a sua revisão o ticket #1655 voutou a apresentar problemas. Pesquisei aqui uma nova solução, voltei a sua revisão [4014] e apliquei esta nova solução. Aqui, no nosso ambiente de desenvolvimento, esta nova solução, revisão [4022], apresentou um bom desempenho e não gerou o problema no ticket #1655.

Aguardando Teste de Alexandre Correia (CELEPAR).

comment:5 Changed 8 years ago by alexandrecorreia

Roberto !!

Isto deixa a abertura da caixa lenta, pois o range é muito grande !!!

$sort_array_msg = $this-> get_msgs($folder, $sort_box_type, $search_box_type, $sort_box_reverse,$msg_range_begin,$num_msgs);

Está em negrito, anteriormente o range da mensagem era limitado pelos parâmetros msg_range_begin e msg_range_end, parametros que são passados para este método.

Desculpe pelo commit numa codificação diferente !!!

comment:6 Changed 8 years ago by roberto.santosjunior

Observe que estou usando agora a função imap_mailboxmsginfo que deixa o desempenho bem melhor. Teste essa versão em seu ambiente.

Com o reverse a codigicação já está ok, so precisamos fechar estes dois problemas(desempenho e pesquisa correta das informações).

Apos os testes comente o que achou.

comment:7 Changed 8 years ago by alexandrecorreia

Roberto !!

Vc chegou a importar na sua caixa aquele zip que enviei pra vc com 1400 mensagens ?

comment:8 Changed 8 years ago by roberto.santosjunior

Não recebi o e-mail com o zip das mensagens, por favor envie para roberto.santosjunior@….

comment:9 Changed 8 years ago by alexandrecorreia

Roberto estou colocando o zip aqui !!

Changed 8 years ago by alexandrecorreia

Mensagens para teste, este arquivo tem aproximadamente 1500 mensagens.

comment:10 Changed 8 years ago by alexandrecorreia

Roberto !!

Arquivo em formato tar.bz2 !!

comment:11 Changed 8 years ago by roberto.santosjunior

Alexandre,

testei em uma pasta acentuada com 2400 mensagens na VM de desenvolvimento, realmente a sua alteração gerou um resultado 4x mais rapida que a solução que implementei. Como podemos solucionar tambem o problema nas pastas acentuadas, referente ao ticket #1655?

Com a sua revisão [4014] temos um resultado rapido mais ainda incorreto para pastas acentuadas.

comment:12 Changed 8 years ago by alexandrecorreia

Opa Roberto !!

Tendo como ponto de partida aquela implementação que deixa mais rápido a abertura da caixa, podiamos a partir dele resolver o problema da acentuação !!! Qual o nome que vc está criando para as caixas ?

comment:13 Changed 8 years ago by roberto.santosjunior

A minha pasta tem o nome de "Teste de Acentuação" e o erro ocorre ao clicar nos links "Não Lidas", "Lidas", "Respondidas" e "Importantes". quando voce clica nos links o contador de mensagens apresenta null ou não mostra um numeral.

comment:14 Changed 8 years ago by alexandrecorreia

Opa Roberto !!

Acabei de reproduzir o problema aqui !! Vou resolver !

comment:15 Changed 8 years ago by alexandrecorreia

  • Status changed from new to closed
  • Resolution set to fixed

Revisão [4038]

comment:16 follow-up: ↓ 17 Changed 8 years ago by roberto.santosjunior

Foi criada uma nova revisão, definitiva, que garante a solução deste ticket e do ticket #1655. Revisão[4150]

comment:17 in reply to: ↑ 16 Changed 8 years ago by rafaelraymundo

Com a correção [4150], a pré-visualização das mensagens no campo assunto passou a não funcionar.

Replying to roberto.santosjunior:

Foi criada uma nova revisão, definitiva, que garante a solução deste ticket e do ticket #1655. Revisão[4150]

comment:18 Changed 8 years ago by roberto.santosjunior

Foi criado um ticket #1805 para corrigir o problema.

Note: See TracTickets for help on using tickets.