[
https://issues.apache.org/jira/browse/TAJO-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14641796#comment-14641796
]
ASF GitHub Bot commented on TAJO-1681:
--------------------------------------
Github user jihoonson commented on the pull request:
https://github.com/apache/tajo/pull/628#issuecomment-124909940
I got your point. As you said, the dump result doesn't have to be verified
since this ticket is related to fix wrong null checking on database name.
If so, how about adding an error message when the given database name is
wrong? I mean, ```isAcceptableDumpingDatabase()``` returns ```false```, it
would be better to print a message to notify users that something is going
wrong. In addition, we can verify error handling with that message.
> Fix TajoDump invalid null check for database name
> -------------------------------------------------
>
> Key: TAJO-1681
> URL: https://issues.apache.org/jira/browse/TAJO-1681
> Project: Tajo
> Issue Type: Bug
> Reporter: DaeMyung Kang
> Assignee: DaeMyung Kang
>
> TajoDump invalid null check in isAcceptableDumpingDatabase
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
