From commits-return-18171-archive-asf-public=cust-asf.ponee.io@pulsar.apache.org Tue Nov 27 03:31:34 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 807CE180677 for ; Tue, 27 Nov 2018 03:31:34 +0100 (CET) Received: (qmail 72420 invoked by uid 500); 27 Nov 2018 02:31:33 -0000 Mailing-List: contact commits-help@pulsar.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@pulsar.apache.org Delivered-To: mailing list commits@pulsar.apache.org Received: (qmail 72411 invoked by uid 99); 27 Nov 2018 02:31:33 -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; Tue, 27 Nov 2018 02:31:33 +0000 From: GitBox To: commits@pulsar.apache.org Subject: [GitHub] sijie closed pull request #3064: Check for nullness of SchemaInfo Message-ID: <154328589298.15801.4171339820169364455.gitbox@gitbox.apache.org> Date: Tue, 27 Nov 2018 02:31:32 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit sijie closed pull request #3064: Check for nullness of SchemaInfo URL: https://github.com/apache/pulsar/pull/3064 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java index 1a9cedc0ab..76c4f53d8d 100644 --- a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java +++ b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java @@ -543,7 +543,7 @@ public void connectionOpened(final ClientCnx cnx) { } SchemaInfo si = schema.getSchemaInfo(); - if (SchemaType.BYTES == si.getType()) { + if (si != null && SchemaType.BYTES == si.getType()) { // don't set schema for Schema.BYTES si = null; } ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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 With regards, Apache Git Services