Mailing-List: contact issues-help@carbondata.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@carbondata.apache.org
From: jackylk <git@git.apache.org>
To: issues@carbondata.apache.org
Reply-To: issues@carbondata.apache.org
References: <git-pr-1739-carbondata@git.apache.org>
In-Reply-To: <git-pr-1739-carbondata@git.apache.org>
Subject: [GitHub] carbondata pull request #1739: [CARBONDATA-1834] when running insert overwri...
Content-Type: text/plain
Message-Id: <20171229080522.D6FEADFE46@git1-us-west.apache.org>
Date: Fri, 29 Dec 2017 08:05:22 +0000 (UTC)
archived-at: Fri, 29 Dec 2017 08:05:24 -0000

Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1739#discussion_r159034322
  
    --- Diff: integration/spark2/src/main/java/org/apache/carbondata/spark/vectorreader/VectorizedCarbonRecordReader.java ---
    @@ -121,12 +126,24 @@ public VectorizedCarbonRecordReader(QueryModel queryModel, InputMetricsStats inp
           while (ext != null) {
             if (ext instanceof FileNotFoundException) {
               throw new InterruptedException(
    -              e.getMessage() + ". insert overwrite may be in progress.Please check");
    +              "Insert overwrite may be in progress.Please check " + e.getMessage());
             }
             ext = ext.getCause();
           }
           throw new InterruptedException(e.getMessage());
         }
    +    catch (Exception e) {
    +      Throwable ext = e;
    +      while (ext != null) {
    +        if (ext instanceof FileNotFoundException) {
    +          LOGGER.error(e);
    +          throw new InterruptedException(
    +              "Insert overwrite may be in progress.Please check " + e.getMessage());
    --- End diff --
    
    use new InterruptedException(msg, e) to add e itself to the InterruptedException object


---