You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oodt.apache.org by "Ricky Nguyen (Created) (JIRA)" <ji...@apache.org> on 2012/03/22 21:08:23 UTC

[jira] [Created] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

filemgr query throws "String index out of range: -1" when some products have undefined metadata values
------------------------------------------------------------------------------------------------------

                 Key: OODT-427
                 URL: https://issues.apache.org/jira/browse/OODT-427
             Project: OODT
          Issue Type: Bug
          Components: file manager
    Affects Versions: 0.3
            Reporter: Ricky Nguyen
             Fix For: 0.4


To reproduce:

1) ingest a ProductType without metadata (except ProductType defined)
2) run a sql query without output format
3) String index out of bounds thrown as shown below

[vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13235978#comment-13235978 ] 

jiraposter@reviews.apache.org commented on OODT-427:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4451/
-----------------------------------------------------------

Review request for Chris Mattmann, brian Foster, Paul Ramirez, Sheryl John, and Thomas Bennett.


Summary
-------

Cleaned up metadata to string methods of QueryResult. If metadata was empty, it was trying to delete a trailing comma that doesn't exist.


This addresses bug OODT-427.
    https://issues.apache.org/jira/browse/OODT-427


Diffs
-----

  trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java 1303205 
  trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java 1303205 

Diff: https://reviews.apache.org/r/4451/diff


Testing
-------

Added unit test to check for empty metadata.

Changed existing unit test to account for implementation-dependent ordering of keys.


Thanks,

Ricky


                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13237119#comment-13237119 ] 

jiraposter@reviews.apache.org commented on OODT-427:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4451/
-----------------------------------------------------------

(Updated 2012-03-23 21:02:03.638390)


Review request for oodt, Chris Mattmann, brian Foster, Paul Ramirez, Sheryl John, and Thomas Bennett.


Changes
-------

1) updated tests
2) added QueryResult.equals() (eclipse auto generated)


Summary
-------

Cleaned up metadata to string methods of QueryResult. If metadata was empty, it was trying to delete a trailing comma that doesn't exist.


This addresses bug OODT-427.
    https://issues.apache.org/jira/browse/OODT-427


Diffs (updated)
-----

  trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java 1303205 
  trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java 1303205 
  trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java PRE-CREATION 

Diff: https://reviews.apache.org/r/4451/diff


Testing
-------

Added unit test to check for empty metadata.

Changed existing unit test to account for implementation-dependent ordering of keys.


Thanks,

Ricky


                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13237120#comment-13237120 ] 

jiraposter@reviews.apache.org commented on OODT-427:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4451/#review6310
-----------------------------------------------------------



trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java
<https://reviews.apache.org/r/4451/#comment13697>

    multi-value met is comma-separated too, so I removed expected values and test for remaining delimiter.



trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java
<https://reviews.apache.org/r/4451/#comment13698>

    multi-value met is comma-separated too, so I removed expected values and test for remaining delimiter.



trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java
<https://reviews.apache.org/r/4451/#comment13699>

    here is the test bfoster suggested


- Ricky


On 2012-03-23 21:02:03, Ricky Nguyen wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4451/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-03-23 21:02:03)
bq.  
bq.  
bq.  Review request for oodt, Chris Mattmann, brian Foster, Paul Ramirez, Sheryl John, and Thomas Bennett.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Cleaned up metadata to string methods of QueryResult. If metadata was empty, it was trying to delete a trailing comma that doesn't exist.
bq.  
bq.  
bq.  This addresses bug OODT-427.
bq.      https://issues.apache.org/jira/browse/OODT-427
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java 1303205 
bq.    trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java 1303205 
bq.    trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4451/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Added unit test to check for empty metadata.
bq.  
bq.  Changed existing unit test to account for implementation-dependent ordering of keys.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ricky
bq.  
bq.


                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13236187#comment-13236187 ] 

jiraposter@reviews.apache.org commented on OODT-427:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4451/
-----------------------------------------------------------

(Updated 2012-03-22 23:47:07.464893)


Review request for oodt, Chris Mattmann, brian Foster, Paul Ramirez, Sheryl John, and Thomas Bennett.


Changes
-------

decided to write unit tests for QueryResult, instead of adding a test to TestSqlQueryCliAction


Summary
-------

Cleaned up metadata to string methods of QueryResult. If metadata was empty, it was trying to delete a trailing comma that doesn't exist.


This addresses bug OODT-427.
    https://issues.apache.org/jira/browse/OODT-427


Diffs (updated)
-----

  trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java 1303205 
  trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java 1303205 
  trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java PRE-CREATION 

Diff: https://reviews.apache.org/r/4451/diff


Testing
-------

Added unit test to check for empty metadata.

Changed existing unit test to account for implementation-dependent ordering of keys.


Thanks,

Ricky


                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13237639#comment-13237639 ] 

jiraposter@reviews.apache.org commented on OODT-427:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4451/#review6325
-----------------------------------------------------------

Ship it!


LGTM

- brian


On 2012-03-23 21:02:03, Ricky Nguyen wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4451/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-03-23 21:02:03)
bq.  
bq.  
bq.  Review request for oodt, Chris Mattmann, brian Foster, Paul Ramirez, Sheryl John, and Thomas Bennett.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Cleaned up metadata to string methods of QueryResult. If metadata was empty, it was trying to delete a trailing comma that doesn't exist.
bq.  
bq.  
bq.  This addresses bug OODT-427.
bq.      https://issues.apache.org/jira/browse/OODT-427
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java 1303205 
bq.    trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java 1303205 
bq.    trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4451/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Added unit test to check for empty metadata.
bq.  
bq.  Changed existing unit test to account for implementation-dependent ordering of keys.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ricky
bq.  
bq.


                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13236423#comment-13236423 ] 

jiraposter@reviews.apache.org commented on OODT-427:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4451/#review6278
-----------------------------------------------------------


Overall great fix!... Just a few comments


trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java
<https://reviews.apache.org/r/4451/#comment13620>

    Just an FYI... you can do this instead here:
    List<String> outputString = Lists.newArrayList();
    
    I'll leave it up to you though if you want to change it or leave it... I just think the Lists class is cool... There are also Sets and Maps classes which do similar



trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java
<https://reviews.apache.org/r/4451/#comment13619>

    This doesn't make sure that nothing else is being return in the value... For example this test would pass for:
    data.dat,Bob,Billy,Bogus
    
    I can see why you changed it, but maybe split it into a list then verify size and then that the list contain data.dat and Bob,Billy



trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java
<https://reviews.apache.org/r/4451/#comment13621>

    Same thing goes here as comment for line 129 in TestSqlQueryCliAction


- brian


On 2012-03-22 23:47:07, Ricky Nguyen wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4451/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-03-22 23:47:07)
bq.  
bq.  
bq.  Review request for oodt, Chris Mattmann, brian Foster, Paul Ramirez, Sheryl John, and Thomas Bennett.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Cleaned up metadata to string methods of QueryResult. If metadata was empty, it was trying to delete a trailing comma that doesn't exist.
bq.  
bq.  
bq.  This addresses bug OODT-427.
bq.      https://issues.apache.org/jira/browse/OODT-427
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java 1303205 
bq.    trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java 1303205 
bq.    trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4451/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Added unit test to check for empty metadata.
bq.  
bq.  Changed existing unit test to account for implementation-dependent ordering of keys.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ricky
bq.  
bq.


                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Resolved] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "Ricky Nguyen (Resolved) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ricky Nguyen resolved OODT-427.
-------------------------------

    Resolution: Fixed

Committed revision 1305427.
                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13238106#comment-13238106 ] 

jiraposter@reviews.apache.org commented on OODT-427:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4451/#review6344
-----------------------------------------------------------

Ship it!


LGTM!


trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java
<https://reviews.apache.org/r/4451/#comment13771>

    Nice.


- Chris


On 2012-03-23 21:02:03, Ricky Nguyen wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4451/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-03-23 21:02:03)
bq.  
bq.  
bq.  Review request for oodt, Chris Mattmann, brian Foster, Paul Ramirez, Sheryl John, and Thomas Bennett.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Cleaned up metadata to string methods of QueryResult. If metadata was empty, it was trying to delete a trailing comma that doesn't exist.
bq.  
bq.  
bq.  This addresses bug OODT-427.
bq.      https://issues.apache.org/jira/browse/OODT-427
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java 1303205 
bq.    trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java 1303205 
bq.    trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/structs/query/TestQueryResult.java PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4451/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Added unit test to check for empty metadata.
bq.  
bq.  Changed existing unit test to account for implementation-dependent ordering of keys.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ricky
bq.  
bq.


                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (OODT-427) filemgr query throws "String index out of range: -1" when some products have undefined metadata values

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OODT-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13236142#comment-13236142 ] 

jiraposter@reviews.apache.org commented on OODT-427:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4451/
-----------------------------------------------------------

(Updated 2012-03-22 22:55:46.864916)


Review request for oodt, Chris Mattmann, brian Foster, Paul Ramirez, Sheryl John, and Thomas Bennett.


Summary
-------

Cleaned up metadata to string methods of QueryResult. If metadata was empty, it was trying to delete a trailing comma that doesn't exist.


This addresses bug OODT-427.
    https://issues.apache.org/jira/browse/OODT-427


Diffs
-----

  trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/structs/query/QueryResult.java 1303205 
  trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/cli/action/TestSqlQueryCliAction.java 1303205 

Diff: https://reviews.apache.org/r/4451/diff


Testing
-------

Added unit test to check for empty metadata.

Changed existing unit test to account for implementation-dependent ordering of keys.


Thanks,

Ricky


                
> filemgr query throws "String index out of range: -1" when some products have undefined metadata values
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OODT-427
>                 URL: https://issues.apache.org/jira/browse/OODT-427
>             Project: OODT
>          Issue Type: Bug
>          Components: file manager
>    Affects Versions: 0.3
>            Reporter: Ricky Nguyen
>             Fix For: 0.4
>
>
> To reproduce:
> 1) ingest a ProductType without metadata (except ProductType defined)
> 2) run a sql query without output format
> 3) String index out of bounds thrown as shown below
> [vpicu@palmer bin]$ ./filemgr-client -u $FILEMGR_URL -op -sql -q 'SELECT EpisodeStartTime FROM Events'
> Mar 22, 2012 12:17:54 PM org.apache.oodt.cas.filemgr.system.XmlRpcFileManagerClient <init>
> INFO: Loading File Manager Configuration Properties from: [../etc/filemgr.properties]
> ERROR: Failed to perform sql query : sortBy 'null', outputFormat 'null', and delimiter '
> ', filterAlgor 'null', startDateTimeMetKey 'null', endDateTimeMetKey 'null', priorityMetKey 'null', null' : String index out of range: -1

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira