Как улучшить производительность GZIP

Question

В настоящее время у меня есть проблема, что этот кусок кода будет называться> 500 тыс. Раз. Размер сжатого byte[] составляет менее 1 КБ. Каждый раз, когда вызывается метод, все потоки должны быть созданы. Поэтому я ищу способ улучшить этот код.

private byte[] unzip(byte[] data) throws IOException, DataFormatException { 

    byte[] unzipData = new byte[4096]; 

    try (ByteArrayInputStream in = new ByteArrayInputStream(data); 
     GZIPInputStream gzipIn = new GZIPInputStream(in); 
     ByteArrayOutputStream out = new ByteArrayOutputStream()) { 

     int read = 0; 
     while((read = gzipIn.read(unzipData)) != -1) { 
      out.write(unzipData, 0, read); 
     } 

     return out.toByteArray(); 
    } 
} 

Я уже tried заменить ByteArrayOutputStream с ByteBuffer, но в момент создания я не знаю, сколько байт мне нужно выделить.

Кроме того, я пытался использовать Inflater, но я наткнулся на описанную проблему here.

Любая другая идея, что я мог бы сделать, чтобы улучшить производительность этого кода.

Answer 1

1. Профиль вашего приложения, чтобы быть уверенным в том, что вы действительно проводите оптимизированное время в этой функции. Неважно, сколько раз вы называете эту функцию; если на него не приходится значительную долю общего времени выполнения программы, тогда оптимизация будет потрачена впустую.

2. Предварительный размер ByteArrayOutputStream. Размер буфера по умолчанию - 32 байта, а для изменения размера требуется копирование всех существующих байтов. Если вы знаете, что ваши декодированные массивы будут около 1k, используйте new ByteArrayOutputStream(2048).

3. Вместо чтения байта за раз, прочитайте блок за раз, используя предварительно выделенный byte[]. Опасайтесь, что вы должны использовать возвращаемое значение от read в качестве входного сигнала для write. Лучше, используйте что-то вроде Jakarta Commons IOUtils.copy(), чтобы избежать ошибок.