Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Support creating ODBC app servers #94

Closed
dmcassel opened this issue May 17, 2013 · 5 comments
Closed

Support creating ODBC app servers #94

dmcassel opened this issue May 17, 2013 · 5 comments
Assignees

Comments

@dmcassel
Copy link
Collaborator

Update the deployer so that it can create ODBC application servers.

@dmcassel
Copy link
Collaborator Author

I the process of doing this feature, I came across a bug: when setup.xqy gets sent to the server, "+" gets turned into " ". Looks like a url-decode somewhere along the way. This is a problem because of lines like this:

declare variable $system-roles as xs:string+ :=

becomes

declare variable $system-roles as xs:string  :=

and

fn:format-number(xs:int(fn:replace($current, "^(\d+)\..*", "$1")), "000"), (: major :)

becomes

fn:format-number(xs:int(fn:replace($current, "^(\d )\..*", "$1")), "000"), (: major :)

@paxtonhare
Copy link
Contributor

That's nasty. Awesome catch.

On May 20, 2013, at 10:36 AM, David Cassel notifications@github.com wrote:

I the process of doing this feature, I came across a bug: when setup.xqy gets sent to the server, "+" gets turned into " ". Looks like a url-decode somewhere along the way. This is a problem because of lines like this:

declare variable $system-roles as xs:string+ :=
becomes

declare variable $system-roles as xs:string :=
and

fn:format-number(xs:int(fn:replace($current, "^(\d+)..*", "$1")), "000"), (: major :)
becomes

fn:format-number(xs:int(fn:replace($current, "^(\d )..*", "$1")), "000"), (: major :)

Reply to this email directly or view it on GitHub.

@dmcassel
Copy link
Collaborator Author

It would be more awesome if I knew how to fix it.

@dmcassel
Copy link
Collaborator Author

Reminder to self: remember to take care of ODBC app servers in wipe

@dmcassel
Copy link
Collaborator Author

The bug I mentioned above appears to be specific to that version of MarkLogic. I upgraded the server from ML 5.0-1.2 to ML 5.0-5 and the problem went away.

dmcassel added a commit that referenced this issue May 20, 2013
Thanks to Bob Starbird (www.linkedin.com/in/robertstarbird) for providing the bulk of the code.
@ghost ghost assigned dmcassel May 21, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants