You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Chris Goffinet (JIRA)" <ji...@apache.org> on 2009/12/09 22:01:18 UTC

[jira] Created: (CASSANDRA-619) sstable2json makes assumption for keyspace

sstable2json makes assumption for keyspace
------------------------------------------

                 Key: CASSANDRA-619
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
             Project: Cassandra
          Issue Type: Improvement
    Affects Versions: 0.9
            Reporter: Chris Goffinet
            Assignee: Eric Evans
            Priority: Trivial
             Fix For: 0.9


Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (CASSANDRA-619) sstable2json makes assumption for keyspace

Posted by "Eric Evans (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CASSANDRA-619?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790437#action_12790437 ] 

Eric Evans commented on CASSANDRA-619:
--------------------------------------

v2-0001-CASSANDRA-619-use-abspath-to-avoid-NPE-parsing-keyspac.txt makes the sstable path absolute, which should prevent this situation entirely.

> sstable2json makes assumption for keyspace
> ------------------------------------------
>
>                 Key: CASSANDRA-619
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Chris Goffinet
>            Assignee: Eric Evans
>            Priority: Trivial
>             Fix For: 0.5
>
>         Attachments: v1-0001-CASSANDRA-619-clearer-error-message-for-an-NPE-case.txt, v2-0001-CASSANDRA-619-use-abspath-to-avoid-NPE-parsing-keyspac.txt
>
>
> Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CASSANDRA-619) sstable2json makes assumption for keyspace

Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/CASSANDRA-619?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jonathan Ellis updated CASSANDRA-619:
-------------------------------------

    Affects Version/s:     (was: 0.9)
        Fix Version/s:     (was: 0.9)
                       0.5

I don't understand the problem -- where else would you put sstables, than under the keyspace directory?

> sstable2json makes assumption for keyspace
> ------------------------------------------
>
>                 Key: CASSANDRA-619
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Chris Goffinet
>            Assignee: Eric Evans
>            Priority: Trivial
>             Fix For: 0.5
>
>
> Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (CASSANDRA-619) sstable2json makes assumption for keyspace

Posted by "Eric Evans (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CASSANDRA-619?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12788334#action_12788334 ] 

Eric Evans commented on CASSANDRA-619:
--------------------------------------

The problem is that SSTable/SSTableReader assumes that it can suss out the keyspace name from the full path to the sstable file (see parseTableName(String)).

Documenting the import/export utilities on the Operations wiki page is on my short list. Do you think that will suffice?



> sstable2json makes assumption for keyspace
> ------------------------------------------
>
>                 Key: CASSANDRA-619
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Chris Goffinet
>            Assignee: Eric Evans
>            Priority: Trivial
>             Fix For: 0.5
>
>
> Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CASSANDRA-619) sstable2json makes assumption for keyspace

Posted by "Eric Evans (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/CASSANDRA-619?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eric Evans updated CASSANDRA-619:
---------------------------------

    Attachment: v1-0001-CASSANDRA-619-clearer-error-message-for-an-NPE-case.txt

> sstable2json makes assumption for keyspace
> ------------------------------------------
>
>                 Key: CASSANDRA-619
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Chris Goffinet
>            Assignee: Eric Evans
>            Priority: Trivial
>             Fix For: 0.5
>
>         Attachments: v1-0001-CASSANDRA-619-clearer-error-message-for-an-NPE-case.txt
>
>
> Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CASSANDRA-619) sstable2json makes assumption for keyspace

Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/CASSANDRA-619?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jonathan Ellis updated CASSANDRA-619:
-------------------------------------

    Component/s: Tools

> sstable2json makes assumption for keyspace
> ------------------------------------------
>
>                 Key: CASSANDRA-619
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Chris Goffinet
>            Assignee: Eric Evans
>            Priority: Trivial
>             Fix For: 0.5
>
>
> Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (CASSANDRA-619) sstable2json makes assumption for keyspace

Posted by "Chris Goffinet (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CASSANDRA-619?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12788353#action_12788353 ] 

Chris Goffinet commented on CASSANDRA-619:
------------------------------------------

Eric,

That's fine.

> sstable2json makes assumption for keyspace
> ------------------------------------------
>
>                 Key: CASSANDRA-619
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Chris Goffinet
>            Assignee: Eric Evans
>            Priority: Trivial
>             Fix For: 0.5
>
>
> Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (CASSANDRA-619) sstable2json makes assumption for keyspace

Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CASSANDRA-619?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790502#action_12790502 ] 

Jonathan Ellis commented on CASSANDRA-619:
------------------------------------------

+1 v2

> sstable2json makes assumption for keyspace
> ------------------------------------------
>
>                 Key: CASSANDRA-619
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Chris Goffinet
>            Assignee: Eric Evans
>            Priority: Trivial
>             Fix For: 0.5
>
>         Attachments: v1-0001-CASSANDRA-619-clearer-error-message-for-an-NPE-case.txt, v2-0001-CASSANDRA-619-use-abspath-to-avoid-NPE-parsing-keyspac.txt
>
>
> Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CASSANDRA-619) sstable2json makes assumption for keyspace

Posted by "Eric Evans (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/CASSANDRA-619?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eric Evans updated CASSANDRA-619:
---------------------------------

    Attachment: v2-0001-CASSANDRA-619-use-abspath-to-avoid-NPE-parsing-keyspac.txt

> sstable2json makes assumption for keyspace
> ------------------------------------------
>
>                 Key: CASSANDRA-619
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-619
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Chris Goffinet
>            Assignee: Eric Evans
>            Priority: Trivial
>             Fix For: 0.5
>
>         Attachments: v1-0001-CASSANDRA-619-clearer-error-message-for-an-NPE-case.txt, v2-0001-CASSANDRA-619-use-abspath-to-avoid-NPE-parsing-keyspac.txt
>
>
> Right now sstable2json makes the assumption that Keyspace folder is one parent above. When copying SSTables over to another machine to verify some things, it wasn't clear this is how it worked until looking at source code. It kept complaining about NullPointerException when trying to lookup an invalid keyspace. Can we be more explicit in this error message and ability to specify a keyspace name?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.