[
https://issues.apache.org/jira/browse/TAJO-1868?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14903378#comment-14903378
]
ASF GitHub Bot commented on TAJO-1868:
--------------------------------------
Github user hyunsik commented on a diff in the pull request:
https://github.com/apache/tajo/pull/768#discussion_r40139286
--- Diff:
tajo-common/src/main/java/org/apache/tajo/exception/ErrorMessages.java ---
@@ -72,6 +72,7 @@
ADD_MESSAGE(UNDEFINED_INDEX_FOR_TABLE, "index ''%s' does not exist",
1);
ADD_MESSAGE(UNDEFINED_INDEX_FOR_COLUMNS, "index does not exist for
'%s' columns of '%s' table", 2);
ADD_MESSAGE(UNDEFINED_INDEX_NAME, "index name '%s' does not exist", 1);
+ ADD_MESSAGE(UNDEFINED_TABLESPACE_HANDLER, "No tablespace handler for
URI scheme '%s'", 1);
--- End diff --
Almost all error messages of ours show just the fact. One example is
``invalid Session '%s'``, and another example is ``index name '%s' does not
exist``. Users cannot know what they mean actually if they do not have
knowledge about internal. This is because ``index name`` are likely to be
generated automatically and many users do not know what is the session.
Our case is the same. Its fact is that there is no tablespace handler to
support the uri scheme. Your suggested error message ``Please check your table
location uri scheme`` is actually based on some assumption that users may
mistypo the uri. But, it may not be true in some cases. Also, this error
message is inconsistent with others.
> Allow TablespaceManager::get to return unregistered tablespace
> --------------------------------------------------------------
>
> Key: TAJO-1868
> URL: https://issues.apache.org/jira/browse/TAJO-1868
> Project: Tajo
> Issue Type: Improvement
> Reporter: Hyunsik Choi
> Assignee: Hyunsik Choi
> Fix For: 0.11.0
>
>
> This patch allows TablespaceManager::get() to return unregistered tablespace
> by URI.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
