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

Pass query parameters to remoteQueryService.planQuery call #2043

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

drewfarris
Copy link
Collaborator

@drewfarris drewfarris commented Jul 25, 2023

  • Previously, no parameters were passed as a part of the remote call to the /plan endpoint. This meant the receiver did not get any information about the query to plan, resulting in an odd 415 Unsupported Media Type error due to the empty POST body.

* Previously, no parameters were passed. This meant the receiver did not get any information about the query to plan, resulting in an odd `415 Unsupported Media Type` error due to the empty POST body.
@drewfarris drewfarris merged commit 31cece0 into release/version3.41 Jul 25, 2023
3 checks passed
@drewfarris
Copy link
Collaborator Author

Interestingly, this issue was present in the release/version3.41 branch and the integraton branch, but not the release/version5.2 and release/version5.8 branches.

We'll need to be careful when we merge upwards.

hgklohr pushed a commit that referenced this pull request Aug 3, 2023
* Previously, no parameters were passed. This meant the receiver did not get any information about the query to plan, resulting in an odd `415 Unsupported Media Type` error due to the empty POST body.
Conflicts:
	warehouse/query-core/src/main/java/datawave/query/tables/RemoteEventQueryLogic.java
hgklohr pushed a commit that referenced this pull request Aug 3, 2023
* Previously, no parameters were passed. This meant the receiver did not get any information about the query to plan, resulting in an odd `415 Unsupported Media Type` error due to the empty POST body.
Conflicts:
	warehouse/query-core/src/main/java/datawave/query/tables/RemoteEventQueryLogic.java
hgklohr added a commit that referenced this pull request Aug 8, 2023
* Closes #1961 - Use MoreExecutors.newDirectExecutorService() for ListenableFuture callbacks (#1962)

* Closes #1963 - Bump ZK to 3.8.1 (#1965)

* 5.2.1

* 5.2.2-SNAPSHOT

* Date volume chooser (#1347) (#1948)

* Date volume chooser (#1347)

* First pass at date based volume chooser

* Moved to balancers package and renamed package

* Adding missed rename

* Update dist.xml to reflect new package name

* Updates for quickstart and docs to point to new jar

* Updates for accumulo 2.1

* Some minor cleanup

* Updating tests

* Update date chooser with comments.

* Updates for date based volume chooser

---------

Co-authored-by: Adam Lerman awlerma <[email protected]>
Co-authored-by: Adam Lerman <[email protected]>

* 5.2.2

* 5.2.3-SNAPSHOT

* Updated to avoid using a separate a listener executor that requires being shutdown (#1980)

* 5.2.3

* 5.2.4-SNAPSHOT

* Update query limits handling to allow rules based on systemFrom values (#1951)

* 5.2.4

* 5.2.5-SNAPSHOT

* Merge release/version3.41 into release/version5.2 (#2033)

* 5.2.5

* 5.2.6-SNAPSHOT

* Fixes #2039 (#2040)

* Fixes #2039
- Restored ability to configure AccumuloOutputFormat's accumulo client via script
- Cleanup of IngestJob's CBMutationOutputFormatter setup for live ingest

* RE: #2039 - Ensure that parent class configure method is used

* 5.2.6

* 5.2.7-SNAPSHOT

* Fix formatting for AccumuloHelper

* ThreadConfigurableLogger not suppressing logs in query metric writing (#2023) (#2024)

Conflicts:
	web-services/common-util/src/main/java/datawave/webservice/common/logging/ThreadConfigurableLogger.java

* 5.2.7

* 5.2.8-SNAPSHOT

* Pass query parameters to remoteQueryService.planQuery call (#2043)

* Previously, no parameters were passed. This meant the receiver did not get any information about the query to plan, resulting in an odd `415 Unsupported Media Type` error due to the empty POST body.
Conflicts:
	warehouse/query-core/src/main/java/datawave/query/tables/RemoteEventQueryLogic.java

* 5.2.8

* 5.2.9-SNAPSHOT

* Correct merge conflicts

---------

Co-authored-by: Keith Ratcliffe <[email protected]>
Co-authored-by: alerman <[email protected]>
Co-authored-by: Adam Lerman awlerma <[email protected]>
Co-authored-by: Adam Lerman <[email protected]>
Co-authored-by: Ivan Bella <[email protected]>
Co-authored-by: Drew Farris <[email protected]>
Co-authored-by: Bill Oley <[email protected]>
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