Оптимизируем, оптимизируем и еще раз оптимизируем

Как бы я хотел, чтобы для начала этот и подобные посты оптимизировались под RSS.

Ну хоть не про сложение String'ов, и на том спасибо :)

Простите что беспокою вас, но у вас в первом слове в заглавии одна буква пропущена.

private final Pattern pattern = Pattern.compile(«xxx»);
— А почему его нельзя сделать статичным он вроде потокобезопасный?

Про класс Date не согласен. Строгая типизация для того и есть, что бы везде было написано Date, а не long (а в комментарии приписка //Date), а уже пользуетесь стандартным библиотекой, или нестандартной, или свой пишете — ваше дело.

public boolean isValid(Date start, Date end) {
long now = System.currentTimeMillis();
return start.getTime() < now && now < end.getTIme();
}

Оптимизация одного частного случая. Написать тест на разное текущее время или сделать проверку, были ли данные валидны на некий момент, вы уже не сможете. Экономия на спичках здесь может создать нехилую такую проблему. Плюс, чисто семантически странно проверять даты на основе timestamp'а. В любом случае, в итоге всегда получается, что кроме timestamp'а нужно узнать часовой пояс, каким-либо образом отформатировать дату и т.п. И в этом случае лучше везде таскать за собой зависимость на Date now, чем усложнять код хардкодами текущего времени.

Это всё моё имхо, конечно.

key.setDeliveredDateContent(truncateToHours(byPeriodKey.getContentTimestamp()));
key.setDeliveredDateAd(truncateToHours(byPeriodKey.getAdTimestamp()));

Знакомые строки :)

Кстати Escape Analysis сам увидит что объект Date метода не покидает:

    public boolean isValid(Date start, Date end) {
        Date now = new Date();
        return start.before(now) && end.after(now); 
    }

И сам соптимизирует к System.currentTimeMillis();

А код останется чуточку более читаемым.

Хотя я сам, конечно, new Date() уже почти никогда не пишу.

Расскажите в двух словах, как вы профилируете Hadoop-задачи.

1. Проблема Date не в deprecated-методах, а в том, что он mutable и дает соблазн изменить объект, передаваемый в аргументе. Здесь, как правильно заметил автор, пригодится joda. Если посмотреть исходники конструктора Date(), то там всего лишь вызов currentTimeMillis(). Если хотите оптимизировать такой код и вызовов действительно много — вынесите за скобки вызов currentTimeMillis (он по какой-то неведомой причине не такой уж и быстрый), сравнивайте long'и.
2. Про Not Null — сам по себе метод весьма странный, так вообще не стоит писать. А еще есть паттерны immutable и null object (не путать с null).
3. В блоке про Regexp, по всей видимости, после правки стоит убрать упоминание ThreadLocal'а.
4. Пример про truncateToHours по неясной причине просто дату заменяют на дату UTC. А вообще это делается проще: new Date(date.getMillis() / 3600000 * 3600000) (если есть уверенность, что мы работаем в часовом поясе, кратном одному часу, что характерно для большинства задач).
5. Про хеш-коды тоже не согласен. Опять же есть паттерн immutable, особенно важный для объектов, которые кладутся в Set (Map), там в качестве оптимизации можно сделать кеширование hashCode, как это сделано, например, в классе String. А использование HashCodeBuilder — личный выбор разработчика, лишь бы equals->hashCode контракт был сохранен.

P.S. Мне не хватает кармы, но, простите, я бы за статью влепил бы минус.

Про SimpleDateFormat и согласен, и несогласен одновременно.
Действительно, это тяжелый метод, который на практике в нашем случае сжирал существенную долю CPU высоконагруженного веб-сервера.
Но я не соглашусь с тем, что виной тому создание нового SimpleDateFormat каждый раз. Медленным здесь является сам процесс конвертации в строку заданного формата. Я эту проблему решил, написав свой конвертер HttpDate, заточенный под конкретный формат, фигурирующий в HTTP протоколе.

Кстати по поводу hash и equals — eclipse умеет генерировать очень симпотичные перекрытия

Not null:
Здесь тоже можно использовать static :)
Постоянные проверки на null сильно засоряют код.

private static final Item ItemObj=new Item(); // =Item.getEmptyStaticObject();

public Item someMethod() {
    //some logic
    if (something) {
        Item item = new Item();
        fillItem(item);
        return item;
    }else{
       return ItemObj;
    }
}