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

Remove untested and outdated 'azuresql.sql' #3086

Merged
merged 1 commit into from
Apr 19, 2021
Merged

Remove untested and outdated 'azuresql.sql' #3086

merged 1 commit into from
Apr 19, 2021

Conversation

chrzaszcz
Copy link
Member

This file was outdated, unsupported and untested.
The docs were misleading, because they gave an impression that it was up to date and working.

We can bring it back when a need arises and there is a way to test it.

@codecov
Copy link

codecov bot commented Apr 16, 2021

Codecov Report

Merging #3086 (b0d43d1) into master (8cfa47c) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3086      +/-   ##
==========================================
- Coverage   78.84%   78.82%   -0.02%     
==========================================
  Files         377      377              
  Lines       31259    31259              
==========================================
- Hits        24645    24641       -4     
- Misses       6614     6618       +4     
Impacted Files Coverage Δ
src/elasticsearch/mongoose_elasticsearch.erl 76.92% <0.00%> (-7.70%) ⬇️
src/mam/mod_mam_elasticsearch_arch.erl 86.60% <0.00%> (-1.79%) ⬇️
src/pubsub/mod_pubsub_db_mnesia.erl 92.40% <0.00%> (-0.43%) ⬇️
src/ejabberd_sm.erl 75.60% <0.00%> (-0.31%) ⬇️
src/mod_muc_room.erl 77.27% <0.00%> (-0.24%) ⬇️
src/mod_roster.erl 76.56% <0.00%> (-0.24%) ⬇️
src/ejabberd_c2s.erl 89.18% <0.00%> (-0.23%) ⬇️
src/mod_muc_log.erl 77.72% <0.00%> (ø)
src/wpool/mongoose_wpool.erl 82.00% <0.00%> (+1.00%) ⬆️
src/ejabberd_local.erl 64.17% <0.00%> (+1.49%) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8cfa47c...b0d43d1. Read the comment docs.

Copy link
Contributor

@leszke leszke left a comment

Choose a reason for hiding this comment

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

All is good! I noticed only one cosmetic issue, no need to take care of it.

@@ -200,11 +200,11 @@ ALTER DATABASE $name_of_your_db SET READ_COMMITTED_SNAPSHOT ON

The command above may take some time.

Then you need to import the SQL schema from either ``mssql2012.sql`` or ``azuresql.sql`` file depending on which database you are using.
Then you need to import the SQL schema from ``mssql2012.sql``.
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor: I think that single grave accents are enough:

Suggested change
Then you need to import the SQL schema from ``mssql2012.sql``.
Then you need to import the SQL schema from `mssql2012.sql`.

@leszke leszke merged commit b074be8 into master Apr 19, 2021
@leszke leszke deleted the remove-azuresql branch April 19, 2021 15:04
leszke added a commit that referenced this pull request Apr 19, 2021
@leszke leszke added this to the 4.2.0 milestone Apr 19, 2021
leszke added a commit that referenced this pull request Apr 19, 2021
leszke added a commit that referenced this pull request Apr 19, 2021
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.

2 participants