[ 
https://issues.apache.org/jira/browse/HAWQ-271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15086916#comment-15086916
 ] 

ASF GitHub Bot commented on HAWQ-271:
-------------------------------------

Github user huor commented on a diff in the pull request:

    https://github.com/apache/incubator-hawq/pull/251#discussion_r49045335
  
    --- Diff: tools/bin/gppylib/commands/base.py ---
    @@ -697,17 +697,10 @@ def __str__(self):
     
         def run(self,validateAfter=False):
             faultPoint = os.getenv('GP_COMMAND_FAULT_POINT')
    -        print "### DEBUG: ENV[GP_COMMAND_FAULT_POINT] = %s" % (faultPoint 
if faultPoint else "None")
    -        print "### DEBUG: self.name = %s" % ("SomeName" if self.name else 
"None")
             if not faultPoint or (self.name and not 
self.name.startswith(faultPoint)):
    -            print "### DEBUG: EXECUTE name = %s" % ("SomeName" if 
self.name else "None")
    -            print "### DEBUG: EXECUTE cmdStr = %s" % ("SomeCmdStr" if 
self.cmdStr else "None")
    -            print "### DEBUG: EXECUTE context = %s" % ("SomeExecContext" 
if self.exec_context else "None")
    -            print "### DEBUG: EXECUTE remoteHost = %s" % ("SomeRemoteHost" 
if self.remoteHost else "None")
                 self.exec_context.execute(self)
             else:
                 # simulate error
    -            print "### DEBUG: CommandResult"
                 self.results = CommandResult(1,'Fault Injection','Fault 
Injection' ,False,True)
                 print self.results
    --- End diff --
    
    This also needs to be removed:
    print self.results


> Remove external python modules
> ------------------------------
>
>                 Key: HAWQ-271
>                 URL: https://issues.apache.org/jira/browse/HAWQ-271
>             Project: Apache HAWQ
>          Issue Type: Task
>          Components: Command Line Tools
>            Reporter: Radar Lei
>            Assignee: Radar Lei
>             Fix For: 2.0.0-beta-incubating
>
>
> We used some external python modules. These modules should to be removed, and 
> user need to install these modules by themselves.
> For example, Crypto  figleaf   lockfile  paramiko  pg8000  psi  pychecker  
> pygresql  simplejson  unittest2  yaml.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to