Skip to content

Inline JOINs #141

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

Merged
merged 1 commit into from
Feb 20, 2017
Merged

Inline JOINs #141

merged 1 commit into from
Feb 20, 2017

Conversation

bigfoot90
Copy link
Contributor

@bigfoot90 bigfoot90 commented Feb 16, 2017

Before:

JOIN
    tbl2
ON
    c1 = c2
JOIN
    tbl3
ON
    c3 = c4

After:

JOIN tbl2 ON c1 = c2
JOIN tbl3 ON c3 = c4

@codecov-io
Copy link

codecov-io commented Feb 16, 2017

Codecov Report

Merging #141 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #141      +/-   ##
============================================
+ Coverage     99.97%   99.97%   +<.01%     
- Complexity     1606     1614       +8     
============================================
  Files            54       54              
  Lines          3759     3765       +6     
============================================
+ Hits           3758     3764       +6     
  Misses            1        1

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 a79e330...ea60c28. Read the comment docs.

@nijel nijel merged commit baf740a into phpmyadmin:master Feb 20, 2017
@nijel
Copy link
Contributor

nijel commented Feb 20, 2017

Merged, thanks for your contribution!

nijel added a commit that referenced this pull request Feb 20, 2017
Signed-off-by: Michal Čihař <michal@cihar.com>
@nijel nijel self-assigned this Feb 20, 2017
@bigfoot90 bigfoot90 deleted the inline-joins branch February 20, 2017 21:27
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.

3 participants