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
KMeans improvements (CQL, hull attr, CLI opts) #1168
Conversation
@@ -102,14 +107,50 @@ public void run() | |||
queryOptions.setAdapter(featureAdapterIds); | |||
|
|||
// This is required due to some funkiness in GeoWaveInputFormat | |||
queryOptions.getAdaptersArray(inputDataStore.createAdapterStore()); | |||
AdapterStore adapterStore = inputDataStore.createAdapterStore(); | |||
queryOptions.getAdaptersArray(adapterStore); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
really? thats not good if its true
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's an old comment. I have no idea why it's there. I think I split it out just to get a ref to the adapterStore for reuse farther down.
} | ||
} | ||
catch (CQLException e) { | ||
LOGGER.error("Unable to parse CQL: " + cqlFilter); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ideally we'd have a stack trace (ie. add , e);
Not worth disapproving the PR jsut for that.
.getFeatureType() | ||
.getGeometryDescriptor() | ||
.getLocalName(); | ||
Filter filter; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
odd, must have been an artifact of something OBE for these two lines to be separate. iNot worth disapproving the PR just for that.
No description provided.