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