You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The current help information doesn't really make any sense:
-es <url> Elasticsearch URL. Default is app:/connections/direct/localhost.xml
If this is supposed to be the XML config file, then let's just say that. I know there are ways around this using different URIs, but this does not make any sense to most people.
While making this update, let's clean up anything in the output run logs that says "Elasticsearch URL" and replace it with something more meaningful to an end user. "OpenSearch Config Path", "OpenSearch Config URI", etc.
⚔️ Parent Epic / Related Tickets
No response
The text was updated successfully, but these errors were encountered:
-es <url> Elasticsearch URL. Default is app:/connections/direct/localhost.xml
New:
-es <uri> File URI to the configuration to connect to the OpenSearch Registry. For example, file:///home/user/.pds/mcp.xml. Default is app:/connections/direct/localhost.xml
Let's keep it -es because I don't think it is worth introducing a backwards incompatible change. If we really want to button this up, then:
-es <uri> (deprecated) File URI to the configuration to connect to the OpenSearch Registry. For example, file:///home/user/.pds/mcp.xml. Default is app:/connections/direct/localhost.xml
-registry <uri> (deprecated) File URI to the configuration to connect to the OpenSearch Registry. For example, file:///home/user/.pds/mcp.xml. Default is app:/connections/direct/localhost.xml
I am good with option 2 with both -es and -registry (not deprecated though, compared to Jordan's comment above). I like the -registry because we could think of enriching the xml connection file, adding other databases (e.g. graphDB, ...) that harvest would update, if that is a route we want to take.
💡 Description
The current help information doesn't really make any sense:
If this is supposed to be the XML config file, then let's just say that. I know there are ways around this using different URIs, but this does not make any sense to most people.
While making this update, let's clean up anything in the output run logs that says "Elasticsearch URL" and replace it with something more meaningful to an end user. "OpenSearch Config Path", "OpenSearch Config URI", etc.
⚔️ Parent Epic / Related Tickets
No response
The text was updated successfully, but these errors were encountered: