[hawq] branch taoz updated: Revert "HAWQ-1722. Fix core dump due to lock is not released before reporting errors when exceeding MaxAORelSegFileStatus"

[huor]

This is an automated email from the ASF dual-hosted git repository.

huor pushed a commit to branch taoz
in repository https://gitbox.apache.org/repos/asf/hawq.git


The following commit(s) were added to refs/heads/taoz by this push:
     new dc93761  Revert "HAWQ-1722. Fix core dump due to lock is not released 
before reporting errors when exceeding MaxAORelSegFileStatus"
dc93761 is described below

commit dc93761e8bb3866771ad5c5c2732406d1c3a110d
Author: Ruilong Huo <huoruil...@163.com>
AuthorDate: Mon Jun 17 13:54:51 2019 +0800

    Revert "HAWQ-1722. Fix core dump due to lock is not released before 
reporting errors when exceeding MaxAORelSegFileStatus"
    
    This reverts commit 33b5f136dbf984c7cdcbfe571e68b5dbc30f0b9c.
---
 src/backend/access/appendonly/appendonlywriter.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/src/backend/access/appendonly/appendonlywriter.c 
b/src/backend/access/appendonly/appendonlywriter.c
index addd33e..63b6bc2 100644
--- a/src/backend/access/appendonly/appendonlywriter.c
+++ b/src/backend/access/appendonly/appendonlywriter.c
@@ -423,8 +423,6 @@ AORelCreateHashEntry(Oid relid)
                        {
                                pfree(allfsinfoParquet);
 
-                               LWLockRelease(AOSegFileLock);
-
                                ereport(ERROR, (errmsg("cannot open more than 
%d "
                                      "append-only table segment "
                                      "files cocurrently",
@@ -460,8 +458,6 @@ AORelCreateHashEntry(Oid relid)
                        {
                                pfree(allfsinfo);
 
-                               LWLockRelease(AOSegFileLock);
-
                                ereport(ERROR, (errmsg("cannot open more than 
%d "
                                      "append-only table segment "
                                      "files cocurrently",
@@ -1151,7 +1147,6 @@ List *SetSegnoForWrite(List *existing_segnos, Oid relid, 
int segment_num,
                                if (new_status == NEXT_END_OF_LIST)
                                {
                                        LWLockRelease(AOSegFileLock);
-
                                        ereport(ERROR, (errmsg("cannot open 
more than %d append-only table segment files concurrently",
                                                                        
MaxAORelSegFileStatus)));
                                }