Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The daemon ignores all but the last entry in the .merovignian_properties file when spawning mserver5 #6320

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2017-05-16 10:15:01 +0200
From: @kutsurak
To: Merovingian devs <>
Version: 11.25.21 (Dec2016-SP4)

Last updated: 2017-06-12 15:14:45 +0200

Comment 25333

Date: 2017-05-16 10:15:01 +0200
From: @kutsurak

User-Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:53.0) Gecko/20100101 Firefox/53.0
Build Identifier:

The daemon uses the file .merovignian_properties in the db directory to set the arguments passed to the mserver5 process. The daemon ignores all the entries in that file except the last one that passes multiple times to the server.

Reproducible: Always

Steps to Reproduce:

Assuming bash:

  1. monetdbd create /tmp/dbfarm
  2. monetdbd start /tmp/dbfarm
  3. monetdb create db
  4. for i in {1..5}; do monetdb set prop$i=val$i db; done
  5. monetdb start db
  6. ps -ww -fC mserver5

Actual Results:

mserver5
--dbpath=/tmp/dbfarm/db
--set merovingian_uri mapi:monetdb://zarquon.zarniwoop.org:50000/db
--set mapi_open false
--set mapi_port 0
--set mapi_usock /tmp/stable_farm/foo/.mapi.sock
--set monet_vault_key /tmp/dbfarm/db/.vaultkey
--set gdk_nr_threads 4
--set max_clients 64
--set sql_optimizer default_pipe
--set prop5 val5
--set prop5 val5
--set prop5 val5
--set prop5 val5
--set prop5 val5
--set monet_daemon yes

Expected Results:

mserver5
--dbpath=/tmp/dbfarm/db
--set merovingian_uri mapi:monetdb://zarquon.zarniwoop.org:50000/db
--set mapi_open false
--set mapi_port 0
--set mapi_usock /tmp/stable_farm/foo/.mapi.sock
--set monet_vault_key /tmp/dbfarm/db/.vaultkey
--set gdk_nr_threads 4
--set max_clients 64
--set sql_optimizer default_pipe
--set prop1 val1
--set prop2 val2
--set prop3 val3
--set prop4 val4
--set prop5 val5
--set monet_daemon yes

Comment 25335

Date: 2017-05-16 11:17:49 +0200
From: MonetDB Mercurial Repository <>

Changeset f088e35112c5 made by Panagiotis Koutsourakis kutsurak@monetdbsolutions.com in the MonetDB repo, refers to this bug.

For complete details, see http//devmonetdborg/hg/MonetDB?cmd=changeset;node=f088e35112c5

Changeset description:

Make sure we are using ALL properties we just read from the file

This fixes bug #6320
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant