You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bloodhound.apache.org by Apache Bloodhound <de...@bloodhound.apache.org> on 2014/03/05 11:15:08 UTC

[Apache Bloodhound] #787: [patch] mysql support in bloodhound_setup.py

#787: [patch] mysql support in bloodhound_setup.py
-------------------------+--------------------
 Reporter:  JoKoT3       |      Owner:  nobody
     Type:  enhancement  |     Status:  new
 Priority:  major        |  Milestone:
Component:  installer    |    Version:  0.7.0
 Keywords:  mysql        |
-------------------------+--------------------
 Hi,

 Here is a patch to bloodhound_setup.py to support an installation on
 mysql.
 It has been tested with the following installation type :
 - all needed parameters given
 - no parameters given, interactive installation
 - by providing the connectstring

 This is a follow-up to this conversation http://mail-
 archives.apache.org/mod_mbox/bloodhound-user/201403.mbox/browser

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/787>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker

Re: [Apache Bloodhound] #787: [patch] mysql support in bloodhound_setup.py

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#787: [patch] mysql support in bloodhound_setup.py
--------------------------+-----------------------
  Reporter:  JoKoT3       |      Owner:  rjollos
      Type:  enhancement  |     Status:  closed
  Priority:  major        |  Milestone:  Release 8
 Component:  installer    |    Version:  0.7.0
Resolution:  fixed        |   Keywords:  mysql
--------------------------+-----------------------
Changes (by rjollos):

 * status:  review => closed
 * resolution:   => fixed


Comment:

 The patch had a few issues:
  * There were tabs in the Python source code, but we must use only spaces.
 The incorrect indentation broke the installation with the SQLite backend.
  * The default database type was specified as //postgres/mysql/sqlite//,
 but it must be //sqlite//.
  * The installer claimed to exit when an unsupported dbtype was entered,
 but did not actually exit.

 I tested for the same conditions described in comment:description, with:
  * MySQL: server: "5.5.34-0ubuntu0.13.04.1", client: "5.5.34", thread-
 safe: 1
  * MySQLdb: 1.2.3

 ----
 (In [1580702])

 0.8dev: Added MySQL support to install script. Refs #787.

 Initial patch by JoKoT3.

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/787#comment:4>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker

Re: [Apache Bloodhound] #787: [patch] mysql support in bloodhound_setup.py

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#787: [patch] mysql support in bloodhound_setup.py
--------------------------+-----------------------
  Reporter:  JoKoT3       |      Owner:  rjollos
      Type:  enhancement  |     Status:  review
  Priority:  major        |  Milestone:  Release 8
 Component:  installer    |    Version:  0.7.0
Resolution:               |   Keywords:  mysql
--------------------------+-----------------------
Changes (by rjollos):

 * status:  accepted => review


-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/787#comment:3>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker

Re: [Apache Bloodhound] #787: [patch] mysql support in bloodhound_setup.py

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#787: [patch] mysql support in bloodhound_setup.py
--------------------------+----------------------
  Reporter:  JoKoT3       |      Owner:  rjollos
      Type:  enhancement  |     Status:  accepted
  Priority:  major        |  Milestone:  next 0.x
 Component:  installer    |    Version:  0.7.0
Resolution:               |   Keywords:  mysql
--------------------------+----------------------
Changes (by rjollos):

 * owner:  nobody => rjollos
 * status:  new => accepted
 * milestone:   => next 0.x


Comment:

 Thanks for the patch. We might be able to get it into the upcoming
 release. If not, then "Release 9".

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/787#comment:1>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker

Re: [Apache Bloodhound] #787: [patch] mysql support in bloodhound_setup.py

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#787: [patch] mysql support in bloodhound_setup.py
--------------------------+-----------------------
  Reporter:  JoKoT3       |      Owner:  JoKoT3
      Type:  enhancement  |     Status:  closed
  Priority:  major        |  Milestone:  Release 8
 Component:  installer    |    Version:  0.7.0
Resolution:  fixed        |   Keywords:  mysql
--------------------------+-----------------------
Changes (by rjollos):

 * owner:  rjollos => JoKoT3


-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/787#comment:5>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker

Re: [Apache Bloodhound] #787: [patch] mysql support in bloodhound_setup.py

Posted by Apache Bloodhound <de...@bloodhound.apache.org>.
#787: [patch] mysql support in bloodhound_setup.py
--------------------------+-----------------------
  Reporter:  JoKoT3       |      Owner:  rjollos
      Type:  enhancement  |     Status:  accepted
  Priority:  major        |  Milestone:  Release 8
 Component:  installer    |    Version:  0.7.0
Resolution:               |   Keywords:  mysql
--------------------------+-----------------------
Changes (by rjollos):

 * milestone:  next 0.x => Release 8


-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/787#comment:2>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker