You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Mikhail Bautin (JIRA)" <ji...@apache.org> on 2012/05/16 01:31:08 UTC

[jira] [Created] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

Mikhail Bautin created HBASE-6007:
-------------------------------------

             Summary: Make getTableRegions return an empty list if the table does not exist
                 Key: HBASE-6007
                 URL: https://issues.apache.org/jira/browse/HBASE-6007
             Project: HBase
          Issue Type: Bug
            Reporter: Mikhail Bautin
            Priority: Minor
         Attachments: D3243.1.patch

Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.


--
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] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279523#comment-13279523 ] 

Hudson commented on HBASE-6007:
-------------------------------

Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #11 (See [https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/11/])
    [jira] [HBASE-6007] Make getTableRegions return an empty list if the table does not exist

Summary:
Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

Test Plan: Unit tests

Reviewers: kannan, liyintang

Reviewed By: liyintang

CC: stack

Differential Revision: https://reviews.facebook.net/D3243 (Revision 1340310)

     Result = FAILURE
mbautin : 
Files : 
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java

                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Assignee: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

Posted by "Phabricator (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279354#comment-13279354 ] 

Phabricator commented on HBASE-6007:
------------------------------------

mbautin has commented on the revision "[jira] [HBASE-6007] [89-fb] Make getTableRegions return an empty list if the table does not exist".

  Michael: I have committed this into trunk.

REVISION DETAIL
  https://reviews.facebook.net/D3243

BRANCH
  fix_get_table_regions

To: Kannan, Liyin, JIRA, mbautin
Cc: stack

                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Assignee: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279374#comment-13279374 ] 

Hudson commented on HBASE-6007:
-------------------------------

Integrated in HBase-TRUNK #2904 (See [https://builds.apache.org/job/HBase-TRUNK/2904/])
    [jira] [HBASE-6007] Make getTableRegions return an empty list if the table does not exist

Summary:
Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

Test Plan: Unit tests

Reviewers: kannan, liyintang

Reviewed By: liyintang

CC: stack

Differential Revision: https://reviews.facebook.net/D3243 (Revision 1340310)

     Result = FAILURE
mbautin : 
Files : 
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java

                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Assignee: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

Posted by "Phabricator (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279138#comment-13279138 ] 

Phabricator commented on HBASE-6007:
------------------------------------

stack has commented on the revision "[jira] [HBASE-6007] [89-fb] Make getTableRegions return an empty list if the table does not exist".

  +1

REVISION DETAIL
  https://reviews.facebook.net/D3243

To: Kannan, Liyin, JIRA, mbautin
Cc: stack

                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] [Updated] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

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

Mikhail Bautin updated HBASE-6007:
----------------------------------

    Attachment: jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch

Patch committed to trunk.
                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

Posted by "Phabricator (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279356#comment-13279356 ] 

Phabricator commented on HBASE-6007:
------------------------------------

mbautin has closed the revision "[jira] [HBASE-6007] [89-fb] Make getTableRegions return an empty list if the table does not exist".

REVISION DETAIL
  https://reviews.facebook.net/D3243

COMMIT
  https://reviews.facebook.net/rHBASE1340310

To: Kannan, Liyin, JIRA, mbautin
Cc: stack

                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Assignee: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

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

Mikhail Bautin resolved HBASE-6007.
-----------------------------------

    Resolution: Fixed
      Assignee: Mikhail Bautin
    
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Assignee: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

Posted by "Phabricator (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279153#comment-13279153 ] 

Phabricator commented on HBASE-6007:
------------------------------------

Liyin has accepted the revision "[jira] [HBASE-6007] [89-fb] Make getTableRegions return an empty list if the table does not exist".

  +1, Thanks Mikhail !

REVISION DETAIL
  https://reviews.facebook.net/D3243

BRANCH
  fix_get_table_regions

To: Kannan, Liyin, JIRA, mbautin
Cc: stack

                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] [Updated] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

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

Phabricator updated HBASE-6007:
-------------------------------

    Attachment: D3243.1.patch

mbautin requested code review of "[jira] [HBASE-6007] [89-fb] Make getTableRegions return an empty list if the table does not exist".
Reviewers: Kannan, Liyin, JIRA

  Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

  I will add a JIRA number when JIRA is back.


TEST PLAN
  Unit tests

REVISION DETAIL
  https://reviews.facebook.net/D3243

AFFECTED FILES
  src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java

To: JIRA

                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] [Updated] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

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

Mikhail Bautin updated HBASE-6007:
----------------------------------

    Attachment:     (was: HBASE-2625-Avoid-byte-buffer-allocations-when-readin-2012-05-18_17_02_05.patch)
    
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

Posted by "Phabricator (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-6007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279163#comment-13279163 ] 

Phabricator commented on HBASE-6007:
------------------------------------

stack has commented on the revision "[jira] [HBASE-6007] [89-fb] Make getTableRegions return an empty list if the table does not exist".

  Want to apply to trunk to Liyin?

REVISION DETAIL
  https://reviews.facebook.net/D3243

BRANCH
  fix_get_table_regions

To: Kannan, Liyin, JIRA, mbautin
Cc: stack

                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] [Updated] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

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

Mikhail Bautin updated HBASE-6007:
----------------------------------

    Attachment: HBASE-2625-Avoid-byte-buffer-allocations-when-readin-2012-05-18_17_02_05.patch

Patch committed to trunk.
                
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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] [Updated] (HBASE-6007) Make getTableRegions return an empty list if the table does not exist

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

Mikhail Bautin updated HBASE-6007:
----------------------------------

    Comment: was deleted

(was: Patch committed to trunk.)
    
> Make getTableRegions return an empty list if the table does not exist
> ---------------------------------------------------------------------
>
>                 Key: HBASE-6007
>                 URL: https://issues.apache.org/jira/browse/HBASE-6007
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Mikhail Bautin
>            Priority: Minor
>         Attachments: D3243.1.patch, jira-HBASE-6007-Make-getTableRegions-return-an-empty-2012-05-18_17_02_40.patch
>
>
> Making the getTableRegions Thrift API method handle TableNotFoundException and return an empty list in that case. Without this the behavior is dependent on whether an HTable object is present in the thread-local cache in case a table was deleted.

--
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