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

Avoid full cluster scans in Connection.checkClusterName #1195

Merged
merged 1 commit into from
Feb 22, 2019
Merged

Avoid full cluster scans in Connection.checkClusterName #1195

merged 1 commit into from
Feb 22, 2019

Conversation

haaawk
Copy link

@haaawk haaawk commented Feb 21, 2019

Select without a partition key results in full cluster
scan that can hurt the performance of the cluster.
There is an easy solution to the problem by just
using a 'local' partition key for the query.

Signed-off-by: Piotr Jastrzebski piotr@scylladb.com

Select without a partition key results in full cluster
scan that can hurt the performance of the cluster.
There is an easy solution to the problem by just
using a 'local' partition key for the query.

Signed-off-by: Piotr Jastrzebski <piotr@scylladb.com>
Copy link
Contributor

@olim7t olim7t left a comment

Choose a reason for hiding this comment

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

SGTM
I don't think it will change anything about performance, at least with Apache Cassandra. But on the other hand it doesn't hurt either, is more correct from a theoretical standpoint, and we already do it for other system.local queries.

@adutra adutra merged commit 60e6696 into apache:3.x Feb 22, 2019
@haaawk
Copy link
Author

haaawk commented Feb 22, 2019

Thank you - much appreciated.

@adutra
Copy link
Contributor

adutra commented Feb 26, 2019

@haaawk We realized that for some reason our CLA bot did not ask you to sign our CLA. Would you mind signing it for us here: https://cla.datastax.com/ ? If you do this today you might be able to get your contribution shipped with 3.7.0. Thanks!

@haaawk
Copy link
Author

haaawk commented Feb 26, 2019

Hi @adutra I can't sign this unfortunately. Not because I don't want to but because I don't qualify. I'm not able to sign because of section 8. Would you have an agreement for companies? My employer could sign it to ensure the code stays free and open source.

@avikivity would be able to provide better information

@adutra
Copy link
Contributor

adutra commented Feb 26, 2019

I understand. I will check with our legal team and will get back to you when I have an answer. Unfortunately, in the meanwhile we'll have to revert your contribution, but hopefully once this legal problem is settled you will be able to contribute it again. Sorry.

@haaawk
Copy link
Author

haaawk commented Feb 26, 2019

@adutra I understand and hope this can be resolved relatively easily. Sorry for the troubles.

adutra pushed a commit that referenced this pull request Feb 26, 2019
@avikivity
Copy link

To be clear, we have no objections to @haaawk signing the CLA; it's just that it makes no provision for people contributing with the consent their employer.

@adutra
Copy link
Contributor

adutra commented Feb 27, 2019

According to our legal team: "Section 8 requires the author to make a representation that he owns the proposed contribution, and not his employer. If the author is unable to sign because his employer owns the proposed contribution (and thus the copyright), then we need his employer to sign a different CLA. If that's the case, please let me know the name of his employer and we will draft it for him."

@haaawk
Copy link
Author

haaawk commented Feb 27, 2019

This is exactly the case @adutra. The name of my employer is ScyllaDB.

@gmatza
Copy link

gmatza commented Apr 5, 2019

@adutra Hello. I work with @haaawk. We'd like to get that CLA signed. Company name, officially, should be ScyllaDB, Ltd. Any legal docs or further discussion with your Legal can come through my email, 'greg at scylladb.com'

@adutra
Copy link
Contributor

adutra commented Apr 5, 2019

@gmatza as a matter of fact I just got word from our legal team. I will send the special CLA to the address you mentioned above. Thanks!

@haaawk
Copy link
Author

haaawk commented Apr 12, 2019

@adutra my company has signed all the documents. Should we proceed with the change?

@adutra
Copy link
Contributor

adutra commented Apr 12, 2019

@haaawk not just yet :-) the ball is now in the hands of our legal team, the CLA needs to be signed by us as well before we are allowed to proceed. Stay tuned.

@haaawk
Copy link
Author

haaawk commented Apr 12, 2019

@adutra I see. Sorry for jumping the gun.

adutra pushed a commit that referenced this pull request May 27, 2019
Select without a partition key results in full cluster
scan that can hurt the performance of the cluster.
There is an easy solution to the problem by just
using a 'local' partition key for the query.

Signed-off-by: Piotr Jastrzebski <piotr@scylladb.com>
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.

5 participants