[
https://issues.apache.org/jira/browse/TRAFODION-2838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16288168#comment-16288168
]
ASF GitHub Bot commented on TRAFODION-2838:
-------------------------------------------
GitHub user DaveBirdsall opened a pull request:
https://github.com/apache/incubator-trafodion/pull/1337
[TRAFODION-2838] Add error check to NATableDB::get
This will prevent cores in NATable::updateExtTableAttrs when an error
occurs under the NATable constructor for a Hive table that has a Trafodion
external table definition.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/DaveBirdsall/incubator-trafodion Trafodion2838
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-trafodion/pull/1337.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1337
----
----
> Add missing error check in NATableDB::get
> -----------------------------------------
>
> Key: TRAFODION-2838
> URL: https://issues.apache.org/jira/browse/TRAFODION-2838
> Project: Apache Trafodion
> Issue Type: Bug
> Components: sql-cmp
> Affects Versions: 2.3-incubating
> Reporter: David Wayne Birdsall
> Assignee: David Wayne Birdsall
>
> In NATableDB::get, there is a call to the NATable constructor that lacks an
> error check afterward. The NATable constructor calls several methods that
> have error returns, leaving the NATable object having sometimes incomplete
> state. In particular, if createNAFileSets fails, the clustering information
> will be missing. This will cause NATable::updateExtTableAttrs to abend when
> it then dereferences a null pointer.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
