NixicЕсть приложение, работает через хибернейт, соответственно при начале его работы в бд postgresql была создана последовательность hibernate_sequence от которой все инсерты и пляшут.

Не надо пускать присвоение ID на самотёк. А то потом придётся много плакать. Нескоро, как раз когда всё менять будет очень тяжело.
Отдельный метод "дай мне ID", и вызывать его ПЕРЕД вставкой.

NixicAlexey Tomin,

я просто немного не понял, почему всё пущено на самотек, ведь последовательность срабатывает каждый раз...

Проблема будет не в этом. А том, например, что где-то будет надо узнать ID до вставки в БД. Например если есть циклические ссылки. Или логирование хитрое. Да мало ли что.

Или ещё вишенка на торте. Если вы включаете ID в хэш-код и equals (а иногда без этого никак), то после insert объект уже другой. А вставка (и генерация ID) может пройти незаметно для этого закоулка кода. И что тут будет- лучше не знать на опыте.

Nixicв чем могут быть проблемы в дальнейшем?

у этой последовательности максимум это 9223372036854775807
а сейчас текущее значение всего 780000+

Ну это-то мелочь. Как раз предгенерация ID ведёт к "лишнему" расходованию ID - мало ли почему вставка не прошла, а ID-то уже взяли.

llemingВ любом случае при ошибке sequence будет увеличится на 1.

Если ты вставляешь по одной записи за раз- да.
А если надо вставить клубок из 10ка записей, а на первой вставке ошибка, то твой вариант "потратит" 1 ID, а мой- 10.

Petro123NixicСейчас пишу дополнительно приложение,
если много \ разных приложений к БД, то чем глубже назначение ID, тем лучше (триггер самой БД).

Да, да- больше ада!
Ещё раз- когда сущность внезапно меняет свой ID - это нехорошо. Это может вылезти боком.

Petro123Alexey TominЕщё раз- когда сущность внезапно меняет свой ID - это нехорошо. Это может вылезти боком.
с чего она сменит пол ID?
Хибер по ID и определяет сущность.

У тебя в java-коде.
Было entity.getId() == null, а потом вдруг entity.getId() == 123. Вот счастье, да?

Petro123ты полный кейс приведи, если не лень.
С чего у меня так будет?
Давай не путать когда 2-ое приложение лезет в обхход хибера и где назначать счётчик.

Entity entity = new Entity(); //entity.id == null
передали в какой-то другой сервис, тот значение сохранил.
em.insert(entity); // А теперь entity.id != null
передали в тот же сервис, тот сравнивает- значение другое (т.к. id другой).

Petro123Alexey TominEntity entity = new Entity(); //entity.id == null
передали в какой-то другой сервис, тот значение сохранил.
- отсоединять объект от сессии хибера нельзя
- сессия хибера должна быть как можно короче, напр 0,01 сек.
Отсюда твой пример не верен.
- либо закрой сессию и получишь ID.
Либо отсоедини от сессии и тогда объект обезличен.

Фигню пишешь.
Ещё раз- создал объект, передал в метод, который его сохранил, и теперь это другой объект - т.к. всё проверяется по equals, который (вполне возможно) включает ID.

Petro123Передал по ссылке?
Зачем его сравнивать?

Нет, не надо ничего сравнивать. Меньше кода- меньше ошибок. Лучше вообще софт не писать- ошибок и не будет.

llemingсравнивать надо, но зачем сравнивать несохраненную сущность с другой сущностью смысла нет особого.

Код пишется по принципу "меньше знаешь- крепче спишь". Поэтому код бизнес-логики может не знать, что за объект ему передан, откуда он, сохранён ли он.

Банально- оператор редактирует некий сложный объект в памяти. До попытки сохранить ему надо сказать, можно ли его будет сохранить. Для этого надо лезть в БД (иногда код реально такой, что крыша едет- бизнес, блин, требования). Но редактируемый объект надо исключить из выборки. Если есть ID- всё просто. Если нет- уже надо на null проверку лишнюю ставить- а это лишние проблемы.

llemingВсе сущности меняют ID внезапно(метод setID ведь есть) если только ID вы не генерируете на уровне app и не передаете в конструктор.

Я за суррогатный ключ всегда и везде. Как результат- ID никогда не меняется. А присвоение ID- как уже сказал- лучше делать сразу после создания, вызвав метод (который сам бегает в последовательность БД, а в дальнейшем- ещё и кэширует значения, чтобы БД не грузить, беря оттуда по 10 штук, к примеру).

А метод setId у меня всегда проверяет, что id был null (ну или 0, если long), а если не так- RuntimeException по рукам.