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.

Sign up for GitHub

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

Jump to bottom

Remove duplicate updateOperationInput #2425

Closed
GCHQDev404 opened this issue May 17, 2021 · 1 comment · Fixed by #2426
Closed

Remove duplicate updateOperationInput #2425

GCHQDev404 opened this issue May 17, 2021 · 1 comment · Fixed by #2426
Labels
tech-debt Relates to Technical Debt
Milestone
v1.19.0

Comments

@GCHQDev404
Copy link
Contributor

OperationHandlerUtil.updateOperationInput
NamedOperationResolver.updateOperationInput
OperationChainHandler.updateOperationInput
GCHQDev404 added a commit that referenced this issue May 17, 2021
@GCHQDev404
gh-2425 remove duplicate updateOperationInput
bd5006e
GCHQDev404 added a commit that referenced this issue May 17, 2021
@GCHQDev404
Revert "gh-2425 remove duplicate updateOperationInput"
3869278 
This reverts commit bd5006e
GCHQDev404 added a commit that referenced this issue May 17, 2021
@GCHQDev404
gh-2425 remove duplicate updateOperationInput
099965f
@github-actions github-actions bot mentioned this issue May 17, 2021
Merged
@n3101
Copy link

n3101 commented Jun 2, 2021

@GCHQDev404 Is this a breaking change?

GCHQDev404 added a commit that referenced this issue Jun 3, 2021
@GCHQDev404
Merge branch 'develop' into gh-2425-remove-duplicate-upgradeOperation…
1d06666 
…Input
GCHQDev404 added a commit that referenced this issue Jun 3, 2021
@GCHQDev404
gh-2425 remove duplicate updateOperationInput
8c705ee
GCHQDev404 added a commit that referenced this issue Jun 15, 2021
@GCHQDev404
Merge branch 'develop' into gh-2425-remove-duplicate-upgradeOperation…
a7f9e1a 
…Input
GCHQDev404 added a commit that referenced this issue Jun 28, 2021
@GCHQDev404
Merge branch 'develop' into gh-2425-remove-duplicate-upgradeOperation…
5af336f 
…Input
GCHQDev404 added a commit that referenced this issue Aug 2, 2021
@GCHQDev404
Merge branch 'develop' into gh-2425-remove-duplicate-upgradeOperation…
9570de4 
…Input
@n3101 n3101 added this to the v1.19.0 milestone Aug 24, 2021
t92549 added a commit that referenced this issue Aug 27, 2021
@t92549
Merge branch 'develop' into gh-2425-remove-duplicate-upgradeOperation…
9ff4732 
…Input
t92549 added a commit that referenced this issue Aug 27, 2021
@GCHQDev404 @t92549
gh-2425 remove duplicate updateOperationInput (#2426)
c18c39d 
Co-authored-by: GCHQDev404 <gchqdev404@users.noreply.github.com>
Co-authored-by: t92549 <80890692+t92549@users.noreply.github.com>
@t92549 t92549 added the tech-debt Relates to Technical Debt label Aug 27, 2021
GCHQDev404 added a commit that referenced this issue Oct 24, 2022
@GCHQDev404 @t92549 @GCHQDeveloper314
gh-2357 federatedstore federated operation merge into v2 alpha (#2769)
c1517b1 
* gh-2357 FederatedStore FederatedOperation Operation/Graph/Merge

* gh-2357 FederatedStore FederatedOperation Handlers

* gh-2357 FederatedStore FederatedOperation IFederationOperation, Handlers & Tests

* gh-2357 FederatedStore FederatedOperation FederatedStore

* gh-2357 FederatedStore FederatedOperation removal of option KEY_OPERATION_OPTIONS_GRAPH_IDS.

* gh-2357 FederatedStore FederatedOperation GetSchema & GetTraits Changes

* gh-2357 FederatedStore FederatedOperation IFederatedOperation

* gh-2357 FederatedStore FederatedOperation deletion of FederatedOperationChain

* gh-2357 FederatedStore FederatedOperation viewValidation

* gh-2357 FederatedStore FederatedOperation Merging of results

* gh-2357 FederatedStore FederatedOperation Tests, CodeStyle, Refactoring & Comments

* gh-2357 FederatedStore FederatedOperation.v02 Builder IO focus

* gh-2357 FederatedStore FederatedOperation.v02. Removing suspected unrequired tests.

* gh-2357 FederatedStore FederatedOperation.v02. Resolving TODOs

* gh-2357 FederatedStore FederatedOperation.v02. Removing overly constraining MIDPUT generic

* gh-2357 FederatedStore FederatedOperation.v02. Resolving TODOs

* gh-2357 FederatedStore FederatedOperation.v02. checkstyle

* gh-2357 FederatedStore FederatedOperation.v02. Resolving TODOs.

* gh-2357 FederatedStore FederatedOperation.v02 with errors

* gh-2357 FederatedStore FederatedOperation.v02 isRequestingAdmin added to IFederation

* gh-2357 FederatedStore FederatedOperation.v02 IFederation builder changes.

* gh-2357 FederatedStore FederatedOperation.v02 IFederation shallowClone changes.

* gh-2357 FederatedStore FederatedOperation.v02 IFederation userRequestingAdminUsage changes.

* gh-2357 FederatedStore FederatedOperation.v02 IFederation userRequestingAdminUsage rename.

* gh-2357 FederatedStore FederatedOperation.v02 bugfix.

* gh-2357 FederatedStore FederatedOperation.v02 removing TODOs.

* gh-2422 FederatedStore ChangeGraphId & ChangeGraphAccess update cache + Test

* gh-2357 FederatedStore FederatedOperation.v02 removing TODOs.

* gh-2357 FederatedStore FederatedOperation.v02 removing TODOs. BugHunt

* gh-2357 FederatedStore FederatedOperation.v02 removing TODOs.

* gh-2425 remove duplicate updateOperationInput

* gh-2357 FederatedStore FederatedOperation.v02 removing TODOs.

* gh-2357 FederatedStore FederatedOperation.v02 FederatedOperation is InputOutput

* gh-2357 FederatedStore FederatedOperation.v02 removing TODOs. BugHunt

* gh-2357 FederatedStore FederatedOperation.v02 FederatedOperation is InputOutput

* fixing bug

* gh-2357 FederatedStore FederatedOperation.v02 FederatedOperation is InputOutput

* gh-2357 FederatedStore FederatedOperation.v02 Error Returning AccumuloAllElementsRetriever

* gh-2357 FederatedStore FederatedOperation.v02 MASSIVE bug fix for IterableConcat use of ChainedIterable Constructor.

* gh-2357 FederatedStore FederatedOperation.v02 IterableConcat fixes

* gh-2357 FederatedStore FederatedOperation.v02 summary

* gh-2357 FederatedStore FederatedOperation.v02 ChainedIterable ported over from Koryphe.
BREAKING CHANGE for any ChainedIterable saved to disc/metal.

* gh-2357 FederatedStore FederatedOperation.v02 checkstyle

* gh-2357 FederatedStore FederatedOperation.v02 Traits

* gh-2455 FederatedStore GetTraits Audit. Removing dead code and correcting tests and behaviours.

* gh-2455 FederatedStore GetTraits Audit. Refactoring FederatedGetTraitsHandler to use FederatedChain. Deprecating Store.getTraits() FederatedGraphStorage.getTraits

* gh-2455 FederatedStore GetTraits Audit. Refactoring FederatedGetTraitsHandler to use FederatedChain. Fixing graphSizeCount.

* gh-2455 FederatedStore GetTraits Audit. Refactoring

* gh-2422 FederatedStore ChangeGraphId & ChangeGraphAccess recovery info.

* gh-2357 FederatedStore FederatedOperation.v02 GetSchema

* gh-2357 FederatedStore FederatedOperation.v02 TODOs

* gh-2357 FederatedStore FederatedOperation.v02 TODOs

* gh-2357 FederatedStore FederatedOperation.v02 TODOs

* gh-2357 FederatedStore FederatedOperation.v02 PR review

* gh-2357 FederatedStore FederatedOperation.v02 Demo fix

* gh-2357 FederatedStore FederatedOperation merge with Alpha1 fixes including a temporary class while Korphye is being released.

* gh-2357 FederatedStore checkstyle

* gh-2369 Remove ChainedIterable Gaffer duplicate

* gh-2369 Dependency upgrade Koryphe 2.2.0

* gh-2369 Deleting Duplicate Koryphe classes

* gh-2369 Deleting Duplicate Koryphe classes updating imports

* Updated Koryphe version to 2.2.0

* gh-2369 corrections

* gh-2369 corrections

* Fix copyright dates

* gh-2369 import fix

* gh-2357 spotless changes

* gh-2357 FederatedStore FederatedOperation merging.

* gh-2357 FederatedStore FederatedOperation temporary classes in lieu of Koryphe changes. see gh-272

* gh-2357 FederatedStore FederatedOperation javadoc

* gh-2357 FederatedStore FederatedOperation default merge function change.

* gh-2357 FederatedStore FederatedOperation FederatedOutputIterableHandler rename and change

* Updated Koryphe version to 2.3.0

* gh-2357 FederatedStore FederatedOperation default merge function change.

* gh-2357 FederatedStore FederatedOperation tidy

* gh-2357 FederatedStore FederatedOperation Test improvement

* gh-2357 FederatedStore deprecated

* gh-2357 FederatedStore checkstyle

* gh-2357 FederatedStore TDD DefaultMerge of GetElements with View Aggregation

* gh-2357 FederatedStore spotless

* gh-2357 FederatedStore DefaultMergeFunction from util.

* gh-2357 FederatedStore DefaultMergeFunction best efforts of concat.

* FederatedStore Alpha4 Team Review PR

* gh-2357 FederatedStore undoing changes out of scope of gh-2357

* gh-2357 FederatedStore spotless checkstyle

* gh-2357 FederatedStore undoing changes out of scope of gh-2357 and spotless checkstyle

* gh-2357 FederatedStore reviewing of Todos, graphIds as List not just CSV.

* gh-2357 FederatedStore reviewing of Todos

* PR merge hot fix

* PR merge hot fix

* PR hotfix formatting

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 PR changes.

* gh-2357 Remove ToIterable, fix bracket typo

* gh-2357 PR changes.

* gh-2357 Fix regex

Co-authored-by: GCHQDev404 <gchqdev404@users.noreply.github.com>
Co-authored-by: t92549 <80890692+t92549@users.noreply.github.com>
Co-authored-by: GCHQDeveloper314 <94527357+GCHQDeveloper314@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tech-debt Relates to Technical Debt
Projects
None yet
Milestone
v1.19.0
Development

Successfully merging a pull request may close this issue.

gh-2425 remove duplicate updateOperationInput gchq/Gaffer
3 participants
@GCHQDev404 @n3101 @t92549