Skip to content

Commit 45a2a57

Browse files
author
jbai
committed
use a constant to store the urn prefix length
1 parent c82c117 commit 45a2a57

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

web/app/dao/DatasetRowMapper.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ public class DatasetRowMapper implements RowMapper<Dataset>
4141
public static String DATASET_OWNER_NAME_COLUMN = "owner_name";
4242
public static String SCHEMA_HISTORY_ID_COLUMN = "schema_history_id";
4343
public static String HDFS_PREFIX = "hdfs";
44+
public static int URN_PREFIX_LEN = 7;
4445

4546

4647
@Override
@@ -102,7 +103,7 @@ public Dataset mapRow(ResultSet rs, int rowNum) throws SQLException
102103
if (dataset.urn.substring(0, 4).equalsIgnoreCase(HDFS_PREFIX))
103104
{
104105
dataset.hdfsBrowserLink = Play.application().configuration().getString(DatasetsDAO.HDFS_BROWSER_URL_KEY) +
105-
dataset.urn.substring(6);
106+
dataset.urn.substring(URN_PREFIX_LEN);
106107
}
107108
}
108109
dataset.source = source;

web/app/dao/DatasetWithUserRowMapper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ public Dataset mapRow(ResultSet rs, int rowNum) throws SQLException
106106
if (dataset.urn.substring(0, 4).equalsIgnoreCase(HDFS_PREFIX))
107107
{
108108
dataset.hdfsBrowserLink = Play.application().configuration().getString(DatasetsDAO.HDFS_BROWSER_URL_KEY) +
109-
dataset.urn.substring(6);
109+
dataset.urn.substring(DatasetRowMapper.URN_PREFIX_LEN);
110110
}
111111
}
112112
dataset.source = source;

web/app/dao/DatasetsDAO.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -454,7 +454,8 @@ public ObjectNode doInTransaction(TransactionStatus status) {
454454
{
455455
if (ds.urn.substring(0, 4).equalsIgnoreCase(DatasetRowMapper.HDFS_PREFIX))
456456
{
457-
ds.hdfsBrowserLink = Play.application().configuration().getString(HDFS_BROWSER_URL_KEY) + ds.urn.substring(7);
457+
ds.hdfsBrowserLink = Play.application().configuration().getString(HDFS_BROWSER_URL_KEY) +
458+
ds.urn.substring(DatasetRowMapper.URN_PREFIX_LEN);
458459
}
459460
}
460461
if (favoriteId != null && favoriteId > 0)

0 commit comments

Comments
 (0)