Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1l1SRG-0005oL-Ei for pgadmin-hackers@arkaria.postgresql.org; Mon, 18 Jan 2021 11:11:14 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1l1SRF-0000Hr-Bg for pgadmin-hackers@arkaria.postgresql.org; Mon, 18 Jan 2021 11:11:13 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1l1SRF-0000Hk-6n for pgadmin-hackers@lists.postgresql.org; Mon, 18 Jan 2021 11:11:13 +0000 Received: from mail-io1-xd32.google.com ([2607:f8b0:4864:20::d32]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1l1SRC-00010Q-TL for pgadmin-hackers@postgresql.org; Mon, 18 Jan 2021 11:11:12 +0000 Received: by mail-io1-xd32.google.com with SMTP id u17so32053369iow.1 for ; Mon, 18 Jan 2021 03:11:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=enterprisedb-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=pQYJYBIl7aiiMNxuBnB4xu6BWhrB1JhT6jYDf56mR6s=; b=0jVrP99TwNJoTqk4+Ykhs4A1r8HyW9PH4inYPhHOhYAlZX96vMvHTu8uUJHNTzCJ8S frUMunGVyh/6ptZfoQrT7cvKCcLTLClSWfwXwGGT0KFckQIwDE21crzrBjxs374MRWkJ V0B2+3hREGQaGDkRxkrmdVDaANbAVWuoTKvoCyOtWIGmaCzthwjzZqlvx1gi/CpZaFRg /rPmrNZdSZMeM8S1SWju9RJoPZ7EZOh2aEyOxW7VwgrJW+26JLNvjZzI8wS4ufp6Es4U BYf8ew4bdS/ULbZroD4RonkW6d2PcNjeWEAmnDyYQs1+gixf5bPD4PfWepnU3125pSfO H3tg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=pQYJYBIl7aiiMNxuBnB4xu6BWhrB1JhT6jYDf56mR6s=; b=WsPTiRJI34fS8IRFQyGGACjtCaCaWVOtt/n9nRO45jLbpdMEgCYJf9tNkEzbZ8WJVj gL6ykcZ2kUoUYBmD9ODFYCp2oZnFjnzMSQ4CvpnJbFNlIbDYNg2dQ5q35DXPmBtGzt2q vn9Y+7vMLJSS7O7vT/DE/2NN7X09hlfpyXg29uHibRrTfMNMcqzNMGNt1EJcQyjxzUXt ztBIcROqX1tE6FlLyp/knyaz2aDBsmPdHFIXq5iTJJwbm0txYha+2Higyuu56Q//xBzY o4+zwrtxuqJgTTw893+86H8pBvDfGnY32xTDB+BSjJv5XRPSwDRcirXYyfwWeOHPGDOX hvYg== X-Gm-Message-State: AOAM530c3k0X3DzlDlX9v3ypRO/CI0EuC6+cekU2okgBjhwjYse2OTCk Am42+twgTLIZju5PPHiuhyi3owG3feO4ryhyd7aA3akZwe3kX0qYHbvX+hWwHOslNzK30dtse3o X8ELOBz1ei6/Ft8PSYDZNznEMZscC85JSgnCC/D3ecfytIo5TQRmrcsnBqIB5AGE8i9vAEg9Jeq LwJFMA1Th2Eq4VztHJPYLtU3yWN/tVjWkOKUD42wCvMCLaV5B2TI/EmX/c2XKHmIl33g== X-Google-Smtp-Source: ABdhPJxQzlSSdDf/VXiCQyigcU4JOC6ldTHNsGGR+UFoTqivLTLIzAKfI8ZI8ieqx4i+mYf/e68w2gx53tGlWyuAz50= X-Received: by 2002:a92:cd81:: with SMTP id r1mr20762862ilb.252.1610968269464; Mon, 18 Jan 2021 03:11:09 -0800 (PST) MIME-Version: 1.0 References: <436f7edcc84e463fac5ed65866dc5ee3@G08CNEXMBPEKD06.g08.fujitsu.local> In-Reply-To: <436f7edcc84e463fac5ed65866dc5ee3@G08CNEXMBPEKD06.g08.fujitsu.local> From: Akshay Joshi Date: Mon, 18 Jan 2021 16:40:58 +0530 Message-ID: Subject: Re: There is an error when checking the major version of servers in schema diff To: "Huang, Jun" Cc: "pgadmin-hackers@postgresql.org" Content-Type: multipart/alternative; boundary="00000000000072421305b92ac73a" X-CLOUD-SEC-AV-Info: enterprisedb,google_mail,monitor X-CLOUD-SEC-AV-Sent: true X-Gm-Spam: 0 X-Gm-Phishy: 0 List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk --00000000000072421305b92ac73a Content-Type: text/plain; charset="UTF-8" Thanks, patch applied. On Sun, Jan 17, 2021 at 6:28 PM Huang, Jun wrote: > Hi, > > The function get_round_val() outputs the wrong value when major version is > before 10. > For example, > Source server version: 90500 > Target server version: 90502 > Or > Source server version: 90502 > Target server version: 90602 > > The attached patch shows the changes. > -- > Thanks && regards > HuangJ > > > > -- *Thanks & Regards* *Akshay Joshi* *pgAdmin Hacker | Principal Software Architect* *EDB Postgres * *Mobile: +91 976-788-8246* --00000000000072421305b92ac73a Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Thanks, patch applied.

On Sun, Jan 17, 2021 at 6:28 PM = Huang, Jun <huangj.fnst@cn= .fujitsu.com> wrote:
Hi,

The function get_round_val() outputs the wrong value when major version is = before 10.
For example,
=C2=A0 =C2=A0Source server version: 90500
=C2=A0 =C2=A0Target server version: 90502
Or
=C2=A0 =C2=A0Source server version: 90502
=C2=A0 =C2=A0Target server version: 90602

The attached patch shows the changes.
--
Thanks && regards
HuangJ





--
Thanks & Regards
Akshay Joshi
pgAdmi= n Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246

--00000000000072421305b92ac73a--