Skip to content

Commit

Permalink
Merge branch 'develop' into 9810-unf6 #9810
Browse files Browse the repository at this point in the history
  • Loading branch information
pdurbin committed Sep 5, 2023
2 parents f891ee3 + 9499d75 commit c4d614d
Show file tree
Hide file tree
Showing 5 changed files with 29 additions and 4 deletions.
6 changes: 5 additions & 1 deletion doc/release-notes/9260-solr930.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ We assume that you already have a user called "solr" (from the instructions abov
```
su - solr
cd /usr/local/solr
wget https://archive.apache.org/dist/lucene/solr/9.3.0/solr-9.3.0.tgz
wget https://archive.apache.org/dist/solr/solr/9.3.0/solr-9.3.0.tgz
tar xvzf solr-9.3.0.tgz
cd solr-9.3.0
cp -r server/solr/configsets/_default server/solr/collection1
Expand All @@ -25,6 +25,10 @@ We assume that you already have a user called "solr" (from the instructions abov
cp /tmp/dvinstall/solrconfig.xml /usr/local/solr/solr-9.3.0/server/solr/collection1/conf
```

1. A Dataverse installation requires a change to the jetty.xml file that ships with Solr.

Edit /usr/local/solr/solr-9.3.0/server/etc/jetty.xml , increasing `requestHeaderSize` from `8192` to `102400`

1. Tell Solr to create the core "collection1" on startup.

```
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@
<dependency>
<groupId>com.google.guava</groupId>
<artifactId>guava</artifactId>
<version>29.0-jre</version>
<version>32.1.2-jre</version>
<type>jar</type>
</dependency>

Expand Down
14 changes: 14 additions & 0 deletions src/test/java/edu/harvard/iq/dataverse/api/AdminIT.java
Original file line number Diff line number Diff line change
Expand Up @@ -734,6 +734,13 @@ public void testMigrateHDLToDOI() {
.statusCode(OK.getStatusCode());
}

/**
* Disabled because once there are new fields in the database that Solr
* doesn't know about, dataset creation could be prevented, or at least
* subsequent search operations could fail because the dataset can't be
* indexed.
*/
@Disabled
@Test
public void testLoadMetadataBlock_NoErrorPath() {
Response createUser = UtilIT.createRandomUser();
Expand Down Expand Up @@ -778,6 +785,13 @@ public void testLoadMetadataBlock_NoErrorPath() {
assertEquals(244, (int) statistics.get("Controlled Vocabulary"));
}

/**
* Disabled because once there are new fields in the database that Solr
* doesn't know about, dataset creation could be prevented, or at least
* subsequent search operations could fail because the dataset can't be
* indexed.
*/
@Disabled
@Test
public void testLoadMetadataBlock_ErrorHandling() {
Response createUser = UtilIT.createRandomUser();
Expand Down
2 changes: 1 addition & 1 deletion src/test/java/edu/harvard/iq/dataverse/api/MoveIT.java
Original file line number Diff line number Diff line change
Expand Up @@ -300,7 +300,7 @@ public void testMoveLinkedDataset() {
.statusCode(OK.getStatusCode())
.body("feed.entry[0].id", CoreMatchers.endsWith(datasetPid));

UtilIT.sleepForReindex(datasetPid, superuserApiToken, 10);
UtilIT.sleepForReindex(datasetPid, superuserApiToken, 20);
Response getLinksAfter = UtilIT.getDatasetLinks(datasetPid, superuserApiToken);
getLinksAfter.prettyPrint();
getLinksAfter.then().assertThat()
Expand Down
9 changes: 8 additions & 1 deletion src/test/java/edu/harvard/iq/dataverse/api/UtilIT.java
Original file line number Diff line number Diff line change
Expand Up @@ -563,7 +563,14 @@ static Response updateDatasetPIDMetadata(String persistentId, String apiToken)
.post("/api/datasets/:persistentId/modifyRegistrationMetadata/?persistentId=" + persistentId);
return response;
}


/**
* Deprecated because once there are new fields in the database that Solr
* doesn't know about, dataset creation could be prevented, or at least
* subsequent search operations could fail because the dataset can't be
* indexed.
*/
@Deprecated
static Response loadMetadataBlock(String apiToken, byte[] body) {
return given()
.header(API_TOKEN_HTTP_HEADER, apiToken)
Expand Down

0 comments on commit c4d614d

Please sign in to comment.