Skip to content
This repository was archived by the owner on Aug 23, 2021. It is now read-only.

Monetdb #183

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Monetdb #183

wants to merge 5 commits into from

Conversation

KimMoriarty
Copy link

  1. Slightly modified and added ddls for monetdb for Wikipedia benchmark.
  2. Modified Wikipedia benchmark to bypass an issue.
  3. Config file for each benchmark: tpcc, tpch, Wikipedia.

Copy link
Member

@apavlo apavlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address my comments.

<driver>com.mysql.jdbc.Driver</driver>
<DBUrl>jdbc:mysql://localhost:3306/tpcc</DBUrl>
<username>root</username>
<dbtype>MONETDB</dbtype>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not change the default sample config file.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Create new config xml files named tpcc_config_monetdb, tpch_config_monetdb, wikipedia_config_monetdb.

}
batchSize++;
} // FOR (revision)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What changed here? Is it just formatting? What parts are MonetDB specific?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We tried to remove the loop for num_revised, which effectively the same as setting num_revised=1 in all cases. We do this because we found otherwise duplicate tuples will be inserted and causing a primary key violation error. We suspect Monetdb may not have a good support for transaction, but we are not sure...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants