@@ -29,12 +29,12 @@ import java.io.Serializable;
  */
 public interface HistoryEntry extends Serializable {
   /**
-   * @return the timestamp in milliseconds of the change
+   * Returns the timestamp in milliseconds of the change.
    */
   long timestampMillis();
 
   /**
-   * @return ID of the new current snapshot
+   * Returns iD of the new current snapshot.
    */
   long snapshotId();
 }
/*
 * Licensed to the Apache Software Foundation (ASF) under one
 * or more contributor license agreements.  See the NOTICE file
 * distributed with this work for additional information
 * regarding copyright ownership.  The ASF licenses this file
 * to you under the Apache License, Version 2.0 (the
 * "License"); you may not use this file except in compliance
 * with the License.  You may obtain a copy of the License at
 *
 *   http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing,
 * software distributed under the License is distributed on an
 * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 * KIND, either express or implied.  See the License for the
 * specific language governing permissions and limitations
 * under the License.
 */

package org.apache.iceberg;

import java.io.Serializable;

/**
 * Table history entry.
 * <p>
 * An entry contains a change to the table state. At the given timestamp, the current snapshot was
 * set to the given snapshot ID.
 */
public interface HistoryEntry extends Serializable {
  /**
   * @return the timestamp in milliseconds of the change
   */
  long timestampMillis();

  /**
   * @return ID of the new current snapshot
   */
  long snapshotId();
}

Was this meant to be `id` or `ID`?