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

Update _177.sql #28

Closed
wants to merge 646 commits into from
Closed

Update _177.sql #28

wants to merge 646 commits into from

Conversation

@CodeToGoodLife
Copy link

@CodeToGoodLife CodeToGoodLife commented Dec 19, 2018

No description provided.

@fishercoder1534
Copy link
Owner

@fishercoder1534 fishercoder1534 commented Dec 19, 2018

Any reasoning for why this change?
I don't see significant performance improvement by this.
Thanks.

@CodeToGoodLife
Copy link
Author

@CodeToGoodLife CodeToGoodLife commented Dec 19, 2018

added GROUP BY Salary.
so new codes can pass by more test cases in leetcode

@fishercoder1534
Copy link
Owner

@fishercoder1534 fishercoder1534 commented Dec 19, 2018

Which new test cases?
I see the original solution is still AC'ed in Leetcode. :)

@CodeToGoodLife
Copy link
Author

@CodeToGoodLife CodeToGoodLife commented Dec 21, 2018

The status of submitted solution shouldn't be changed after new test cases are added

@fishercoder1534
Copy link
Owner

@fishercoder1534 fishercoder1534 commented Dec 21, 2018

I'm confused, what do you mean?

@fishercoder1534 fishercoder1534 force-pushed the fishercoder1534:master branch from e83744e to 11cd4e9 Jun 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.