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

amagyar pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ambari.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 4b247f2  [AMBARI-23384]. PQS start fails after Ambari upgrade due to 
Bad file descriptor (amagyar) (#810)
4b247f2 is described below

commit 4b247f2db3a45c1f6ec59fb768167d2291115fe5
Author: Attila Magyar <[email protected]>
AuthorDate: Thu Mar 29 08:33:19 2018 +0200

    [AMBARI-23384]. PQS start fails after Ambari upgrade due to Bad file 
descriptor (amagyar) (#810)
    
    * AMBARI-23384. PQS start fails after Ambari upgrade due to Bad file 
descriptor (amagyar)
    
    * AMBARI-23384. PQS start fails after Ambari upgrade due to Bad file 
descriptor (amagyar)
---
 .../HBASE/0.96.0.2.0/package/scripts/phoenix_queryserver.py          | 5 +++++
 .../HBASE/0.96.0.2.0/package/scripts/phoenix_service.py              | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git 
a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/phoenix_queryserver.py
 
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/phoenix_queryserver.py
index dfc35fb..0828c78 100644
--- 
a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/phoenix_queryserver.py
+++ 
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/phoenix_queryserver.py
@@ -23,6 +23,8 @@ from resource_management.libraries.functions.stack_features 
import check_stack_f
 from resource_management.libraries.script import Script
 from phoenix_service import phoenix_service
 from hbase import hbase
+from resource_management.core.exceptions import Fail
+from resource_management.libraries.functions.decorator import retry
 
 # Note: Phoenix Query Server is only applicable to stack version supporting 
Phoenix.
 class PhoenixQueryServer(Script):
@@ -45,6 +47,9 @@ class PhoenixQueryServer(Script):
     self.configure(env)
     phoenix_service('start')
 
+  @retry(times=3, sleep_time=5, err_class=Fail) # XXX PID file is not always 
created in time. Should be idempotent.
+  def post_start(self, env=None):
+    return super(PhoenixQueryServer, self).post_start(env)
 
   def stop(self, env, upgrade_type=None):
     import params
diff --git 
a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/phoenix_service.py
 
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/phoenix_service.py
index 42d9cd1..3c2890f 100644
--- 
a/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/phoenix_service.py
+++ 
b/ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/phoenix_service.py
@@ -40,7 +40,7 @@ def phoenix_service(action = 'start'): # 'start', 'stop', 
'status'
         Execute(daemon_cmd,
                 user=format("{hbase_user}"),
                 environment=env)
-  
+
       elif action == 'stop':
         Execute(daemon_cmd,
                 user=format("{hbase_user}"),

-- 
To stop receiving notification emails like this one, please contact
[email protected].

Reply via email to