joshelser commented on a change in pull request #339: [ACCUMULO-4751] Set
createdTime for status records
URL: https://github.com/apache/accumulo/pull/339#discussion_r158520718
##########
File path:
test/src/main/java/org/apache/accumulo/test/replication/StatusMakerIT.java
##########
@@ -246,4 +253,97 @@ public void closedMessagesCreateOrderRecords() throws
Exception {
Assert.assertFalse("Found more files unexpectedly",
expectedFiles.hasNext());
Assert.assertFalse("Found more entries in replication table unexpectedly",
iter.hasNext());
}
+
+ @Test
+ public void orderRecordsCreatedWithNoCreatedTime() throws Exception {
+ String sourceTable = testName.getMethodName();
+ conn.tableOperations().create(sourceTable);
+ ReplicationTableUtil.configureMetadataTable(conn, sourceTable);
+
+ BatchWriter bw = conn.createBatchWriter(sourceTable, new
BatchWriterConfig());
+ String walPrefix = "hdfs://localhost:8020/accumulo/wals/tserver+port/";
+ List<String> files = Arrays.asList(walPrefix + UUID.randomUUID(),
walPrefix + UUID.randomUUID(), walPrefix + UUID.randomUUID(),
+ walPrefix + UUID.randomUUID());
+ Map<String,Integer> fileToTableId = new HashMap<>();
+
+ Status.Builder statBuilder =
Status.newBuilder().setBegin(0).setEnd(0).setInfiniteEnd(true).setClosed(true);
+
+ Map<String,Long> statuses = new HashMap<>();
+ int index = 1;
Review comment:
Just make this a `long` and change the `Integer.toString()` to
`Long.toString()`?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
