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

Fix243 245 254 #33

Merged
merged 18 commits into from
Apr 5, 2012
Merged

Fix243 245 254 #33

merged 18 commits into from
Apr 5, 2012

Conversation

jcookems
Copy link
Contributor

Fixes the following issues:

Note: The fix for 254 causes Jersey to log a lot of SEVERE messages. Perhaps someone can suggest a fix for that.

xmlr.close();
}
catch (XMLStreamException e1) {
// TODO: What to throw here?
Copy link

Choose a reason for hiding this comment

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

What to throw here? I think we can just re-throw what is caught here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, will look into doing that.

@jcookems
Copy link
Contributor Author

jcookems commented Apr 5, 2012

Addressed code comments and added unit test.

<p>This SDK allows you to build Windows Azure applications in Java that allow
#Windows Azure SDK for Java

This SDK allows you to build Windows Azure applications in Java that allow
Copy link

Choose a reason for hiding this comment

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

It seems there are two white space between "applications in" and the following "Java". I think one is sufficient.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

jcookems added a commit that referenced this pull request Apr 5, 2012
@jcookems jcookems merged commit bb5fbed into Azure:dev-bookmark Apr 5, 2012
joostdenijs pushed a commit to joostdenijs/azure-sdk-for-java that referenced this pull request Jan 18, 2013
Forward integration from Dev to Job
joostdenijs pushed a commit to joostdenijs/azure-sdk-for-java that referenced this pull request Jan 18, 2013
Added fluent setters for list query params
gcheng pushed a commit that referenced this pull request Apr 12, 2013
jianghaolu pushed a commit to jianghaolu/azure-sdk-for-java that referenced this pull request Apr 5, 2017
jianghaolu pushed a commit to jianghaolu/azure-sdk-for-java that referenced this pull request Apr 5, 2017
Added fluent setters for list query params
jianghaolu pushed a commit to jianghaolu/azure-sdk-for-java that referenced this pull request Apr 5, 2017
lenala pushed a commit to lenala/azure-sdk-for-java that referenced this pull request Nov 16, 2018
Updated with 3-21 keyvault preview-swagger
g2vinay pushed a commit to g2vinay/azure-sdk-for-java that referenced this pull request Mar 4, 2019
g2vinay pushed a commit to g2vinay/azure-sdk-for-java that referenced this pull request May 30, 2019
Set the default maxChunkSize for HttpObjectAggregationConfigurator to 2 MB as CosmosDB allows documents of size upto 2 MB

Fixes Azure#33
samvaity added a commit to samvaity/azure-sdk-for-java that referenced this pull request Dec 19, 2019
navalev added a commit to navalev/azure-sdk-for-java that referenced this pull request Dec 24, 2019
TestCanUseTopAndSkipForClientSidePaging
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.

4 participants