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

removed storage prefix #55

Merged
merged 1 commit into from
Aug 24, 2016
Merged

removed storage prefix #55

merged 1 commit into from
Aug 24, 2016

Conversation

matryer
Copy link
Contributor

@matryer matryer commented Aug 23, 2016

No description provided.

@matryer matryer changed the title removed storage interface removed storage prefix Aug 23, 2016
@jasonhancock
Copy link
Contributor

that means customers are going to need to know to prefix their configurations with storage-, right?

@matryer
Copy link
Contributor Author

matryer commented Aug 23, 2016

Yes - but here's the rub - some places in Oracle have different prefixes Storage vs storage

@jasonhancock
Copy link
Contributor

Shouldn't we just identify the correct prefix to use in each situation?

@matryer
Copy link
Contributor Author

matryer commented Aug 23, 2016

Why is it not just a configuration value!?

@aaronedell
Copy link

Today - there's no way to browse Oracle storage through other tools like Cloudberry without knowing when to use storage- vs. Storage- so I feel like the customer would know this. What I don't know is if there are other possibilities. But the nice thing is that in the Oracle setup web GUI, it gives you the restful API URL which has either Storage- or storage-

@matryer
Copy link
Contributor Author

matryer commented Aug 23, 2016

I think it will only need to change their swiftTenantName

@matryer
Copy link
Contributor Author

matryer commented Aug 23, 2016

@jasonhancock do you know something we don't? (Obviously loads of things, I mean specifically relating to this topic)

@jasonhancock
Copy link
Contributor

@matryer nope...fine by me, but maybe these sorts of things highlight the need for tool tips or additional information popups on the location configuration view.

@dahernan dahernan merged commit 9873fe9 into master Aug 24, 2016
@dahernan dahernan deleted the removeprefix-54 branch August 24, 2016 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants