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 'attribute: huge' from config def file and C++ code. #22900

Closed
wants to merge 118 commits into from
Closed

Remove 'attribute: huge' from config def file and C++ code. #22900

wants to merge 118 commits into from

Conversation

geirst
Copy link
Member

@geirst geirst commented Jun 3, 2022

@toregge please review

gjoranv and others added 30 commits June 2, 2022 21:42
@gjoranv
Build for JDK 17
06af464
@gjoranv
Replace deprecated LogRecord.getThreadID -> getLongThreadID
2c432ef
@gjoranv
Replace deprecated LogRecord.setThreadID -> setLongThreadID
bc166c6
@gjoranv
JDK 17: Use Opcodes.ASM9 for bundle-plugin
b6194ef
@gjoranv
JDK 17: Use Opcodes.ASM9 for abi-check-plugin
da57cc6
@gjoranv
Require jdk17 and maven 3.6
1554e04
@jonmv @gjoranv
Remove DocumentAccess.createDefault
b5eb1f7
@jonmv @gjoranv
Remove compatibility translation Storage --> Content policy
7d55bd8
@jonmv @gjoranv
Remove unused test data
75c82a8
@arnej27959 @gjoranv
just fail when unknown summary class is requested
5bc7626
@arnej27959 @gjoranv
allow null pointer for summary class
18f84b0
@baldersheim @gjoranv
GC compression inside document
aa79258
@arnej27959 @gjoranv
allow empty string
f6b818f
@bjorncs @gjoranv
Remove 'jetty' attribute from container element
56cc45a
@baldersheim @gjoranv
Flip default from xml to json for incomming feed format.
abe4723
@arnej27959 @gjoranv
never use "qrserver" as service name
636ed71
@arnej27959 @gjoranv
update unit tests
6578b56
@bjorncs @gjoranv
Enable filtering 'strict-mode' by default
610b388
@bjorncs @gjoranv
Update default grouping configuration for Vespa 8
daa88ea
@arnej27959 @gjoranv
flip default
52aff6f
@baldersheim @gjoranv
GC deprecated statistics module
dfc7451
@arnej27959 @gjoranv
use IllegalInputException
7627e8a
@bjorncs @gjoranv
Remove deprecated http response metrics
8a94c30
@bjorncs @gjoranv
Remove deprecated APIs exposing Guava types
2f3d88e
@bjorncs @gjoranv
Remove deprecated APIs exposing Jackson types
f9c78b7
@baldersheim @gjoranv
Remove '.sum' form vds sum metrics.
d6b2ec6 
Remove '.sum' from metric names for storage node and also remove the average metrics for the same.
Remove '.sum' from distributor metrics set and remove distributor average metrics.
GC '.sum' from distributor metric names.
Remove '.alldisks' from metric names and update tests.
GC '.alldisks' from filestor metrics.
@arnej27959 @gjoranv
switch default for map and wset rendering
9636f61
@gjoranv
Test new language features from Java 11 to 17
ccd6073 
- Records, switch statements and pattern matching for instanceof.
@bjormel @gjoranv
Rename 'qrs' to 'access'
1fb6c7c
@yngveaasheim @gjoranv
Fix TODOs for Vespa 8.
d154b4c
Harald Musum and others added 18 commits June 2, 2022 23:22
Remove XML schema for removed support for generic service cluster
d207048
@arnej27959
cleanup more deprecated APIs
180f50c
@arnej27959
no PublicApi for events
27cced9
@arnej27959
make some implementation classes non-public
f17602d
@arnej27959
com.yahoo.log.event cleanup
81dfa26
@gjoranv
Do not install Jersey bundles in jdisc.
e0ca48e
@gjoranv
Remove all Jersey related dependencies from vespa_jersey2
7023a7d 
- This artifact will be renamed later.
@gjoranv
Remove workaround to export javax.annotation[.security]
a34b570 
- These exports were required by Jersey bundles.
@gjoranv
Add Vespa 9 TODO to remove javax.inject
cee3a55
@gjoranv
Remove jersey related dependencies from parent poms and enforcers.
6fae6e5
@arnej27959 @baldersheim
rawAsBase64 is now true by default
e41e03c
@arnej27959
remove unused config
f698368
@arnej27959
remove unused config
9cfcd18
@arnej27959 @baldersheim
remove SuppressWarnings no longer needed
b5f2217
@arnej27959 @bratseth
no more tensors in attribute prefetch
a2bd3a6 
* less hardcoded constants
@gjoranv
Do not attempt to install the now non-existent jersey jars.
6e468bc
@bjorncs
Don't provide Jetty bundles from 'container' artifact
813daea 
Add dependency on 'jetty-http' with scope test instead of adding false dependencies with 'container-test'.
@geirst
Remove 'attribute: huge' from config def file and C++ code.
c2f2b94
@geirst geirst requested a review from toregge June 3, 2022 11:18
@geirst
Copy link
Member Author

geirst commented Jun 3, 2022

@gjoranv or @arnej27959 please merge when factory is ok

@geirst geirst mentioned this pull request Jun 3, 2022
Closed
@gjoranv
Copy link
Member

gjoranv commented Jun 3, 2022

Merged in another PR, right @geirst ?

@gjoranv gjoranv closed this Jun 3, 2022
@toregge
Copy link
Member

toregge commented Jun 3, 2022

The original changes in this PR (Remove 'attribute: huge' from config def file and C++ code.) have not been merged.

@toregge toregge reopened this Jun 3, 2022
@gjoranv
Copy link
Member

gjoranv commented Jun 3, 2022

Superseded by #22911

@gjoranv gjoranv closed this Jun 3, 2022
@baldersheim baldersheim deleted the geirst/remove-attribute-huge-config-def branch June 16, 2022 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@toregge toregge toregge approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.