[ 
https://issues.apache.org/jira/browse/HBASE-13300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16305634#comment-16305634
 ] 

Chia-Ping Tsai commented on HBASE-13300:
----------------------------------------

grep our source code. Seems some IA.Public and IA.LP classes still use the 
timeStamp. Would you please fix them before 2.0 release? BTW, the topic need to 
be updated.
{code:title=StoreFile.java}
  /**
   * Get the modification time of this store file. Usually will access the file 
system so throws
   * IOException.
   */
  long getModificationTimeStamp() throws IOException;
{code}

{code:title=ReplicationLoadSink.java}
  public long getTimeStampsOfLastAppliedOp() {
    return this.timeStampsOfLastAppliedOp;
  }
{code}

{code:title=MetricsSource}
  /**
   * Get the timeStampsOfLastShippedOp, if there are multiple groups, return 
the latest one
   * @return lastTimestampForAge
   */
  public long getTimeStampOfLastShippedOp() {
    long lastTimestamp = 0L;
    for (long ts : lastTimeStamps.values()) {
      if (ts > lastTimestamp) {
        lastTimestamp = ts;
      }
    }
    return lastTimestamp;
  }
{code}

> Fix casing in getTimeStamp() and setTimestamp() for Mutations
> -------------------------------------------------------------
>
>                 Key: HBASE-13300
>                 URL: https://issues.apache.org/jira/browse/HBASE-13300
>             Project: HBase
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 1.0.0
>            Reporter: Lars George
>            Assignee: Jan Hentschel
>         Attachments: HBASE-13300.master.001.patch, 
> HBASE-13300.master.002.patch, HBASE-13300.master.003.patch, HBASE-13300.xlsx
>
>
> For some reason we have two ways of writing this method. It should be 
> consistent.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to