From dev-return-62087-archive-asf-public=cust-asf.ponee.io@storm.apache.org Sun Jul 26 02:07:35 2020 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mailroute1-lw-us.apache.org (mailroute1-lw-us.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with ESMTPS id 8B1CD18064D for ; Sun, 26 Jul 2020 04:07:35 +0200 (CEST) Received: from mail.apache.org (localhost [127.0.0.1]) by mailroute1-lw-us.apache.org (ASF Mail Server at mailroute1-lw-us.apache.org) with SMTP id 6C708126D3B for ; Sun, 26 Jul 2020 02:07:32 +0000 (UTC) Received: (qmail 70862 invoked by uid 500); 26 Jul 2020 02:07:31 -0000 Mailing-List: contact dev-help@storm.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@storm.apache.org Delivered-To: mailing list dev@storm.apache.org Received: (qmail 70850 invoked by uid 99); 26 Jul 2020 02:07:30 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 26 Jul 2020 02:07:30 +0000 From: =?utf-8?q?GitBox?= To: dev@storm.apache.org Subject: =?utf-8?q?=5BGitHub=5D_=5Bstorm=5D_Crim_opened_a_new_pull_request_=233314=3A?= =?utf-8?q?_=5BSTORM-3680=5D_Upgrade_Jedis_Library?= Message-ID: Date: Sun, 26 Jul 2020 02:07:30 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Crim opened a new pull request #3314: URL: https://github.com/apache/storm/pull/3314 ## What is the purpose of the change To support future improvements to the storm-redis package, lets upgrade its underlying library (Jedis) it uses to communicate with Redis. Upgrading will clear the way for [STORM-3665](https://issues.apache.org/jira/browse/STORM-3665) as Stream support in the Jedis library was not added until version 3.x ## How was the change tested As no previous test coverage existed for the bolts provided by `storm-redis` I've started to add integration tests using TestContainers. Let me know if this is an acceptable way to provide test coverage and I will build out tests for the remaining bolts. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: users@infra.apache.org