<p></p>
<p><b>@joto</b> commented on this pull request.</p>

<ul>
<li>Maybe use <code>public</code> as default, that would remove some <code>empty()</code> checks?</li>
<li>New <code>--schema</code> option is not documented (should be at least in doc/osm2pgsql.1). It should also be documented in <code>doc/flex.md</code>, that the flex backend doesn't use this option.</li>
<li>Adding more command line options is going in the wrong direction I believe. I'd rather see some kind of config file.</li>
<li>This forces the schema to be the same in the pgsql output and the middle. There is some precedence with the <code>--tablespace*</code> options to allow these to be different. Also it doesn't work on the gazeteer backend and multi-backend (which is deprecated anyway, so that's fine).</li>
<li>No tests (see also <code>tests/test-output-pgsql-schema.cpp</code>).</li>
</ul>
<p>I think we should have a general idea where we are going with the configuration and how we want to handle the options for the different outputs and the middle in a systematic way before adding more options.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/osm2pgsql/pull/1241#pullrequestreview-442294780">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AA6353SGORSFTIJ5XXYDWBLRZWS4NANCNFSM4OPABPQQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AA6353X2AVCOV5YFRUDMP5LRZWS4NA5CNFSM4OPABPQ2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGODJOOD7A.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/osm2pgsql/pull/1241#pullrequestreview-442294780",
"url": "https://github.com/openstreetmap/osm2pgsql/pull/1241#pullrequestreview-442294780",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>